FailedChanges

Summary

  1. [lldb] Add a SymbolFileProvider to record and replay calls to dsymForUUID (details)
  2. [x86][AArch64] adjust fast-math-flags in tests; NFC (details)
  3. [DAGCombine]: Fold X/Sqrt(X) to Sqrt(X) (details)
  4. [LLDB] Fix how ValueObjectVariable handles DW_AT_const_value when the DWARFExpression holds the data that represents a constant value (details)
  5. Fix test for D77924. (details)
  6. [LLDB] Fix SVE offset calculation in NativeRegisterContextLinux_arm64 (details)
  7. [not][test] Fix disable-symbolization.test when 'printenv' is not available (details)
  8. [MLInliner] Support training that doesn't require partial rewards (details)
  9. Properly pass modules flags to frontend when using -std=c++20 instead of -std=c++2a. (details)
Commit a842950b62b6d029a392c3c312c6495d6368c2a4 by Jonas Devlieghere
[lldb] Add a SymbolFileProvider to record and replay calls to dsymForUUID

When replaying a reproducer captured from a core file, we always use
dsymForUUID for the kernel binary. When enabled, we also use it to find
kexts. Since these files are already contained in the reproducer,
there's no reason to call out to an external tool. If the tool returns a
different result, e.g. because the dSYM got garbage collected, it will
break reproducer replay. The SymbolFileProvider solves the issue by
mapping UUIDs to module and symbol paths in the reproducer.

Differential revision: https://reviews.llvm.org/D86389
The file was modifiedlldb/source/Commands/CommandObjectReproducer.cpp
The file was addedlldb/test/Shell/Reproducer/Inputs/core
The file was modifiedlldb/source/Utility/ReproducerProvider.cpp
The file was addedlldb/test/Shell/Reproducer/Inputs/dsymforuuid.sh
The file was modifiedlldb/source/Symbol/LocateSymbolFile.cpp
The file was modifiedlldb/include/lldb/Utility/Reproducer.h
The file was modifiedlldb/include/lldb/Utility/ReproducerProvider.h
The file was modifiedlldb/source/Symbol/LocateSymbolFileMacOSX.cpp
The file was addedlldb/test/Shell/Reproducer/TestDebugSymbols.test
Commit a74dc598fb6b878908ffe769025f79f17e54cab0 by spatel
[x86][AArch64] adjust fast-math-flags in tests; NFC

This goes with the proposal in D86403.
The file was modifiedllvm/test/CodeGen/AArch64/sqrt-fastmath.ll
The file was modifiedllvm/test/CodeGen/X86/sqrt-fastmath.ll
Commit 62e91bf563337fff93be0059ecb1f4dae32d8980 by spatel
[DAGCombine]: Fold X/Sqrt(X) to Sqrt(X)

With FMF ( "nsz" and " reassoc") fold X/Sqrt(X) to Sqrt(X).

This is done after targets have the chance to produce a
reciprocal sqrt estimate sequence because that expansion
is probably more efficient than an expansion of a
non-reciprocal sqrt. That is also why we deferred doing
this transform in IR (D85709).

Differential Revision: https://reviews.llvm.org/D86403
The file was modifiedllvm/test/CodeGen/AArch64/sqrt-fastmath.ll
The file was modifiedllvm/test/CodeGen/X86/sqrt-fastmath.ll
The file was modifiedllvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
Commit 93b255142bb7025f62cf83dd5b7d3b04aab5445b by shafik
[LLDB] Fix how ValueObjectVariable handles DW_AT_const_value when the DWARFExpression holds the data that represents a constant value

In some cases when we have a DW_AT_const_value and the data can be found in the
DWARFExpression then ValueObjectVariable does not handle it properly and we end
up with an extracting data from value failed error.

The test is a very stripped down assembly file since reproducing this relies on the results of compiling with -O1 which may not be stable over time.

Differential Revision: https://reviews.llvm.org/D86311
The file was modifiedlldb/source/Core/ValueObjectVariable.cpp
The file was addedlldb/test/Shell/SymbolFile/DWARF/DW_AT_const_value.s
Commit 96a979c0c2950d118da48ac3307e279a41c5a945 by enh
Fix test for D77924.

The trailing 'L' was missing in the expectation.

Differential Revision: https://reviews.llvm.org/D86321
The file was modifiedlibcxxabi/test/test_demangle.pass.cpp
Commit 4283320b7286dc94367b22df09499dc934e1fbf9 by omair.javaid
[LLDB] Fix SVE offset calculation in NativeRegisterContextLinux_arm64

There was typo left from changes in CalculateSVEOffset where we moved
FPSR/FPCR offset calculation into WriteRegister and ReadRegister.

Differential Revision: https://reviews.llvm.org/D79699
The file was modifiedlldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_arm64.cpp
Commit a5e65ad3a30cd77a768e093276e3426c6d81a32d by maskray
[not][test] Fix disable-symbolization.test when 'printenv' is not available

On Windows, 'env' or 'printenv' may not exist.

Also switch back to 'env' which is specified by POSIX.1-2017. 'printenv' is not
standard (I picked it because 'printenv' exists on GnuWin32 but 'env' does not).

Reviewed By: zequanwu

Differential Revision: https://reviews.llvm.org/D86496
The file was modifiedllvm/test/tools/not/disable-symbolization.test
Commit 8c63df2416dc7a492f6464519b516a202147dc91 by mtrofin
[MLInliner] Support training that doesn't require partial rewards

If we use training algorithms that don't need partial rewards, we don't
need to worry about an ir2native model. In that case, training logs
won't contain a 'delta_size' feature either (since that's the partial
reward).

Differential Revision: https://reviews.llvm.org/D86481
The file was modifiedllvm/lib/Analysis/DevelopmentModeInlineAdvisor.cpp
The file was modifiedllvm/test/Transforms/Inline/ML/development-training-log.ll
Commit 91604949ef2fc8401d17a31362c46c310d5550b6 by richard
Properly pass modules flags to frontend when using -std=c++20 instead of -std=c++2a.

In passing, also teach the driver to map /std:c++latest to -std=c++20
not -std=c++2a.
The file was modifiedclang/test/Driver/modules.cpp
The file was modifiedclang/lib/Driver/ToolChains/Clang.cpp
The file was modifiedclang/test/Driver/cl-options.c