SuccessChanges

Summary

  1. [X86] Directly call EmitTest in two places instead of creating a null (details)
  2. [X86] Make EmitCmp into a static function and explicitly return chain (details)
  3. [LLDB] Fix address computation for inline function (details)
  4. [clang] Fix the canonicalization of paths in (details)
Commit 9b6fafa3990fb5963f0558d13d53abc461fb3f29 by craig.topper
[X86] Directly call EmitTest in two places instead of creating a null
constant and calling EmitCmp. NFCI
EmitCmp will just immediately call EmitTest and discard the null
constant only to have EmitTest create it again if it doesn't fold.
So just skip all that and go directly to EmitTest.
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.cpp
Commit bf507d4259c389cc21d44760b8ebf462edd8e5d0 by craig.topper
[X86] Make EmitCmp into a static function and explicitly return chain
result for STRICT_FCMP. NFCI
The only thing its getting from the X86TargetLowering class is the
subtarget which we can easily pass. This function only has one call site
now since this might help the compiler inline it.
Explicitly return both the flag result and the chain result for
STRICT_FCMP nodes. This removes an assumption in the caller that
getValue(1) is the right way to get the chain.
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.cpp
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.h
Commit 92211bf0f15ba46b5eeb88b7ea580ff539dcdd4e by aclopte
[LLDB] Fix address computation for inline function
Summary: Fixes PR41237 - SIGSEGV on call expression evaluation when
debugging clang
When linking multiple compilation units that define the same functions,
the functions is merged but their debug info is not. This ignores debug
info entries for functions in a non-executable sections; those are
functions that were definitely dropped by the linker.
Reviewers: spyffe, clayborg, jasonmolenda
Reviewed By: clayborg
Subscribers: labath, aprantl, lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D71487
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp
The file was addedlldb/test/Shell/SymbolFile/DWARF/inline-function-address.ll
The file was addedlldb/test/Shell/SymbolFile/DWARF/inline-function-address-shared.test
Commit e8efac4b15303932581c128dc3976f4359388338 by karl-johan.karlsson
[clang] Fix the canonicalization of paths in
-fdiagnostics-absolute-paths
In the current implementation of clang the canonicalization of paths in
diagnostic messages (when using -fdiagnostics-absolute-paths) only works
if the symbolic link is in the directory part of the filename, not if
the file itself is a symbolic link to another file.
This patch adds support to canonicalize the complete path including the
file.
Reviewers: rsmith, hans, rnk, ikudrin
Reviewed By: rnk
Differential Revision: https://reviews.llvm.org/D70527
The file was modifiedclang/lib/Frontend/TextDiagnostic.cpp
The file was modifiedclang/include/clang/Basic/FileManager.h
The file was addedclang/test/Frontend/absolute-paths-symlinks.c
The file was modifiedclang/lib/Basic/FileManager.cpp