SuccessChanges

Summary

  1. [utils] Use llvm::sort (NFC) (details)
  2. [AMDGPU] Use llvm::is_contained (NFC) (details)
  3. [StringExtras] Rename SubsequentDelim to ListSeparator (details)
  4. [NFC] Add -std=c11 to attr-availability.c (details)
  5. [NPM][Inliner] Temporarily remove inline_stats test case for always (details)
  6. AArch64: fix regression introduced by fcmp immediate selection. (details)
Commit a396e2e088eeab974a5d386df9466757a4bdced0 by kazu
[utils] Use llvm::sort (NFC)
The file was modifiedllvm/utils/FileCheck/FileCheck.cpp
The file was modifiedllvm/utils/TableGen/GlobalISel/GIMatchDagInstr.cpp
The file was modifiedllvm/utils/TableGen/GlobalISel/GIMatchDag.cpp
Commit 4707b21298a6daf28b08d9395568ccc94a549711 by kazu
[AMDGPU] Use llvm::is_contained (NFC)
The file was modifiedllvm/lib/Target/AMDGPU/R600ControlFlowFinalizer.cpp
Commit 8fd8ff1f67e4919954162ccf5f0e9b5e9c2d53fb by kazu
[StringExtras] Rename SubsequentDelim to ListSeparator

This patch renames SubsequentDelim to ListSeparator to clarify the
purpose of the class.

Differential Revision: https://reviews.llvm.org/D94649
The file was modifiedllvm/unittests/ADT/StringExtrasTest.cpp
The file was modifiedllvm/include/llvm/ADT/StringExtras.h
The file was modifiedllvm/lib/CodeGen/MachineBasicBlock.cpp
Commit be68c9222b85815612e6bd8bc606a87e3111a0fb by douglas.yung
[NFC] Add -std=c11 to attr-availability.c

This test will fail with any toolchains that don't default to C11.

Adding this switch to the clang invocation in the test fixes the issue.

Patch by Justice Adams!

Reviewed By: dyung

Differential Revision: https://reviews.llvm.org/D94829
The file was modifiedclang/test/Parser/attr-availability.c
Commit a61e42efbb73e55c44cbb0eb2686c7b4a25ca812 by mtrofin
[NPM][Inliner] Temporarily remove inline_stats test case for always
inline

The stats are printed at InlinePass destruction. When we have 2 of them,
it appears the destruction order of the Passes std::vector of the pass
manager differs in msan builds - example:
http://lab.llvm.org:8011/#/builders/74/builds/2135.

This reproes locally, too.

Temporarily removing the sub-test case, to green the build, and will
follow up with a stat dumping alternative that does not depend on vector
element dtor order.
The file was modifiedllvm/test/Transforms/Inline/inline_stats.ll
Commit 8456c3a789285079ad35d146e487436b5a27b027 by Amara Emerson
AArch64: fix regression introduced by fcmp immediate selection.

Forgot to check if the predicate is safe to commutate operands.
The file was modifiedllvm/test/CodeGen/AArch64/GlobalISel/select-fcmp.mir
The file was modifiedllvm/lib/Target/AArch64/GISel/AArch64InstructionSelector.cpp