SuccessChanges

Summary

  1. Remove -DFLANG_BUILD_NEW_DRIVER=ON from builders. NFCI (details)
Commit 1062df17752fb11a207f9cd8dc7854fc73144f0e by diana.picus
Remove -DFLANG_BUILD_NEW_DRIVER=ON from builders. NFCI

FLANG_BUILD_NEW_DRIVER defaults to ON now, so there's no need to add it
anymore.

Differential Revision: https://reviews.llvm.org/D102320
The file was modifiedbuildbot/osuosl/master/config/builders.py (diff)

Summary

  1. [CSSPGO] Update pseudo probe distribution factor based on inline context. (details)
  2. [mlir] Fix in_bounds attr handling in TransferReadPermutationLowering (details)
  3. [mlir] Lower permutation maps on TransferWriteOps (details)
  4. Add support for complex constants to MLIR core. (details)
  5. [clang][deps] NFC: Stop assuming the TU's context hash (details)
  6. [clang][deps] NFC: Report modules' context hash (details)
  7. [PowerPC] add a testcase for reverse memory op; nfc (details)
  8. [clang] NFC: Remove CompilerInstance::getInvocationPtr (details)
  9. [LLD][MinGW] Add --fatal-warnings and --no-fatal-warnings flags (details)
  10. [dexter] Remove unnecessary double check on conditional breakpoints (details)
  11. [dexter] Remove requirement for a condition in DexLimitSteps (details)
  12. [libcxx] NFC. Add missed articles to _LIPCPP_ASSERT messages in <list> (details)
  13. [DAGCombiner] Relax an assertion to an early return (details)
  14. Revert "[NFC][LSAN] Limit the number of concurrent threads is the test" (details)
Commit f28ee1a2b386ad7fa7489cad55840e78849b4abf by hoy
[CSSPGO] Update pseudo probe distribution factor based on inline context.

With prelink inlining, pseudo probes with same ID can come from different inline contexts. Such probes should not share samples and their factors should be fixed up separately.

I'm seeing 0.3% speedup for SPEC2017 overall. Benchmark 631.deepsjeng_s benefits the most, about 4%.

Reviewed By: wenlei, wmi

Differential Revision: https://reviews.llvm.org/D102429
The file was addedllvm/test/Transforms/SampleProfile/pseudo-probe-update-2.ll
The file was modifiedllvm/include/llvm/IR/PseudoProbe.h
The file was modifiedllvm/lib/Transforms/IPO/SampleProfileProbe.cpp
The file was modifiedllvm/include/llvm/Transforms/IPO/SampleProfileProbe.h
Commit 6774e5a995fcc7e1f2360bbeaf8628ae88159430 by springerm
[mlir] Fix in_bounds attr handling in TransferReadPermutationLowering

The in_bounds attribute should also be transposed.

Differential Revision: https://reviews.llvm.org/D102572
The file was modifiedmlir/test/Dialect/Vector/vector-transfer-lowering.mlir
The file was modifiedmlir/lib/Dialect/Vector/VectorTransforms.cpp
The file was addedmlir/test/Integration/Dialect/Vector/CPU/test-transfer-permutation-lowering.mlir
Commit 7ddeffee55766005327abbac85838225069cc164 by springerm
[mlir] Lower permutation maps on TransferWriteOps

Add TransferWritePermutationLowering, which replaces permutation maps of TransferWriteOps with vector.transpose.

Differential Revision: https://reviews.llvm.org/D102548
The file was modifiedmlir/lib/Dialect/Vector/VectorTransforms.cpp
The file was modifiedmlir/test/Dialect/Vector/vector-transfer-lowering.mlir
Commit 5ef21506b98cb5ad5e1280f8d2a7a9fa8bf9b4f8 by akuegel
Add support for complex constants to MLIR core.

BEGIN_PUBLIC
Add support for complex constants to MLIR core.
END_PUBLIC

Differential Revision: https://reviews.llvm.org/D101908
The file was modifiedmlir/test/Dialect/LLVMIR/invalid.mlir
The file was modifiedmlir/include/mlir/Target/LLVMIR/ModuleTranslation.h
The file was modifiedmlir/test/Dialect/Standard/invalid.mlir
The file was modifiedmlir/lib/Dialect/LLVMIR/IR/LLVMDialect.cpp
The file was modifiedmlir/lib/Dialect/Linalg/Transforms/FusionOnTensors.cpp
The file was modifiedmlir/lib/IR/BuiltinAttributes.cpp
The file was modifiedmlir/lib/Dialect/StandardOps/IR/Ops.cpp
The file was modifiedmlir/test/Target/LLVMIR/llvmir.mlir
The file was modifiedmlir/include/mlir/Dialect/StandardOps/IR/Ops.td
The file was modifiedmlir/lib/Target/LLVMIR/ModuleTranslation.cpp
The file was modifiedmlir/test/Dialect/Standard/ops.mlir
The file was modifiedmlir/test/Target/LLVMIR/llvmir-invalid.mlir
Commit b9d5b0c20180e41f24722e622b8c3a3bb510fc1d by Jan Svoboda
[clang][deps] NFC: Stop assuming the TU's context hash

The context hash of modular dependencies can be different from the context hash of the original translation unit if we modify their `CompilerInvocation`s.

Stop assuming the TU's context hash everywhere.

No functionality change here, since we're still currently using the unmodified TU CompilerInvocation to compute the context hash.

Reviewed By: dexonsmith

Differential Revision: https://reviews.llvm.org/D102473
The file was modifiedclang/include/clang/Tooling/DependencyScanning/ModuleDepCollector.h
The file was modifiedclang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp
The file was modifiedclang/lib/Tooling/DependencyScanning/ModuleDepCollector.cpp
Commit d3fb4b9065e93e2617db44f3ab9c31f341d9cd1f by Jan Svoboda
[clang][deps] NFC: Report modules' context hash

This patch eagerly constructs and modifies CompilerInvocation of modular dependencies in order to report the correct context hash instead of the hash of the original translation unit.

No functionality change here, since we currently don't modify CompilerInvocation in a way that affects the context hash.

Depends on D102473.

Reviewed By: dexonsmith

Differential Revision: https://reviews.llvm.org/D102482
The file was modifiedclang/include/clang/Tooling/DependencyScanning/ModuleDepCollector.h
The file was modifiedclang/lib/Tooling/DependencyScanning/ModuleDepCollector.cpp
Commit f6584b85c644fdbffd5e7617a68ceb3f8d140aec by czhengsz
[PowerPC] add a testcase for reverse memory op; nfc
The file was addedllvm/test/CodeGen/PowerPC/vsx-shuffle-le-load.ll
Commit d9cb70ba29dbd74aac294caad0686f1ed52295f9 by Jan Svoboda
[clang] NFC: Remove CompilerInstance::getInvocationPtr

This member function was introduced in 0a92e09c ([clang][deps] Generate the full command-line for modules) in order to keep the CompilerInvocation object alive after CompilerInstance goes out of scope. However, d3fb4b90 ([clang][deps] NFC: Report modules' context hash) removes that use-case, making this function dead.
The file was modifiedclang/include/clang/Frontend/CompilerInstance.h
Commit 84306ef9c442e335b53c3d87948258a528346e6f by martin
[LLD][MinGW] Add --fatal-warnings and --no-fatal-warnings flags

Differential Revision: https://reviews.llvm.org/D102514
The file was modifiedlld/MinGW/Driver.cpp
The file was modifiedlld/MinGW/Options.td
The file was modifiedlld/test/MinGW/driver.test
Commit a11117a7c0a23f37bdee5c611bbaac22e89c775d by orlando.hyams
[dexter] Remove unnecessary double check on conditional breakpoints

Remove the `ConditionalController._conditional_met` method. This was missed in
the recent ConditionalController refactor (D98699). We don't need to check that
the conditions for a conditional breakpoint have been met because
`DebuggerBase.get_triggered_breakpoint_ids` returns the set of ids for
breakpoints which have been triggered.

To get the "triggered breakpoints" from lldb we use `GetStopReasonDataCount`
and `GetStopReasonDataAtIndex`. It seems that these functions count all
breakpoints associated with the location which lldb has stopped at, regardless
of their condition. i.e. Even if we have two breakpoints at the same source
location that have mutually exclusive conditions, both will be found this way
when either condition is true. To get around this, we store a map of breakpoint
{id: condition} `_breakpoint_conditions` and evaluate the conditions of the
triggered breakpoints to filter the set down to those which are unconditional
or have a condition which evaluates to true.

Essentially we are just moving the condition double check from a general
debugger controller into the lldb specific wrapper. This tidy up will help make
upcoming patches simpler.

Reviewed By: chrisjackson

Differential Revision: https://reviews.llvm.org/D101431
The file was modifieddebuginfo-tests/dexter/dex/debugger/lldb/LLDB.py
The file was modifieddebuginfo-tests/dexter/dex/debugger/DebuggerControllers/ConditionalController.py
Commit e35a5492a46d419478daf65eeaed5036cfcfa3e5 by orlando.hyams
[dexter] Remove requirement for a condition in DexLimitSteps

Currently the DexLimitSteps command requires at least one condition. This patch
lets users elide the condition to specify that the breakpoint range should
always be activated when the leading line is stepped on. This patch also
updates the terminology used in the `ConditionalController` class from the
terms 'conditional' and 'unconditional' to 'leading' and 'trailing' when
referring to the breakpoints in the DexLimitSteps range because the leading
breakpoint can now be unconditional.

Reviewed By: chrisjackson

Differential Revision: https://reviews.llvm.org/D101438
The file was modifieddebuginfo-tests/dexter/dex/command/commands/DexLimitSteps.py
The file was modifieddebuginfo-tests/dexter/Commands.md
The file was addeddebuginfo-tests/dexter/feature_tests/subtools/test/err_limit_steps_no_values.cpp
The file was modifieddebuginfo-tests/dexter/dex/debugger/DebuggerControllers/ConditionalController.py
The file was addeddebuginfo-tests/dexter/feature_tests/commands/perfect/limit_steps/unconditional.cpp
Commit a18b5f0188b5929a58790dd0f57c06098d276fd1 by kbessonova
[libcxx] NFC. Add missed articles to _LIPCPP_ASSERT messages in <list>
The file was modifiedlibcxx/include/list
Commit 85e31eddf216181c6b8b26bca760f3395d621f9a by fraser
[DAGCombiner] Relax an assertion to an early return

The select-of-constants transform was asserting that its constant vector
inputs did not implicitly truncate their input without that as an
explicit precondition to the function. This patch relaxes that assertion
into an early return to skip the optimization.

Reviewed By: RKSimon

Differential Revision: https://reviews.llvm.org/D102393
The file was modifiedllvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
The file was addedllvm/test/CodeGen/RISCV/rvv/select-sra.ll
Commit 65936b952964d257c387ce2a7a9362a2ce297a63 by flo
Revert "[NFC][LSAN] Limit the number of concurrent threads is the test"

This reverts commit 2a73b7bd8cf7620fc0e478ac838b07ee6649dd8a.

This appears to be causing the following failures on GreenDragon:
  LeakSanitizer-AddressSanitizer-x86_64 :: TestCases/many_threads_detach.cpp
  LeakSanitizer-Standalone-x86_64 :: TestCases/many_threads_detach.cpp

First failure:
    http://green.lab.llvm.org/green/job/clang-stage1-RA/20754/

Still failing in latest build:
    http://green.lab.llvm.org/green/job/clang-stage1-RA/20928/
The file was modifiedcompiler-rt/test/lsan/TestCases/many_threads_detach.cpp

Summary

  1. Remove -DFLANG_BUILD_NEW_DRIVER=ON from builders. NFCI (details)
Commit 1062df17752fb11a207f9cd8dc7854fc73144f0e by diana.picus
Remove -DFLANG_BUILD_NEW_DRIVER=ON from builders. NFCI

FLANG_BUILD_NEW_DRIVER defaults to ON now, so there's no need to add it
anymore.

Differential Revision: https://reviews.llvm.org/D102320
The file was modifiedbuildbot/osuosl/master/config/builders.py