SuccessChanges

Summary

  1. [PhaseOrdering] add test for hoisting/CSE (PR46115); NFC (details)
  2. [utils] change update_test_checks.py use of 'TMP' value names (details)
  3. AArch64/GlobalISel: Fix incorrect ptrmask usage for alignment (details)
  4. clang-tidy and clang-query wont crash with invalid command line options (details)
  5. [utils] update expected strings in tests; NFC (details)
  6. Revert "clang-tidy and clang-query wont crash with invalid command line options" (details)
Commit 91b45fb527af8f1f83b51e7238642d3af011bad4 by spatel
[PhaseOrdering] add test for hoisting/CSE (PR46115); NFC
The file was addedllvm/test/Transforms/PhaseOrdering/X86/vdiv.ll
Commit bfdc2552664d6f0bb332a9c6a115877020f3c1df by spatel
[utils] change update_test_checks.py use of 'TMP' value names

As discussed in PR45951:
https://bugs.llvm.org/show_bug.cgi?id=45951

There's a potential name collision between update_test_checks.py and -instnamer
and/or manually-generated IR test files because all of them try to use the
variable name that should never be used: "tmp".

This patch proposes to reduce the odds of collision and adds a warning if we
detect the problem. This will cause regression test churn when regenerating
CHECK lines on existing files.

Differential Revision: https://reviews.llvm.org/D80584
The file was modifiedllvm/utils/UpdateTestChecks/common.py (diff)
Commit 95f65a7c6cebba7dbcd955bc02235f5d3581ff44 by arsenm2
AArch64/GlobalISel: Fix incorrect ptrmask usage for alignment

I inverted the mask when I ported to the new form of G_PTRMASK in
8bc03d2168241f7b12265e9cd7e4eb7655709f34.

I don't think this really broke anything, since G_VASTART isn't
handled for types with an alignment higher than the stack alignment.
The file was modifiedllvm/lib/CodeGen/GlobalISel/MachineIRBuilder.cpp (diff)
The file was modifiedllvm/test/CodeGen/AArch64/GlobalISel/legalize-vaarg.mir (diff)
Commit f23ddbe3c3ae5f40b99ba272afc3d16b800ba8b9 by n.james93
clang-tidy and clang-query wont crash with invalid command line options

Summary: Motivated by [[ https://bugs.llvm.org/show_bug.cgi?id=46141 | clang-tidy crashed for unknown command line argument. ]]

Reviewers: aaron.ballman, alexfh

Reviewed By: aaron.ballman

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D80879
The file was modifiedclang-tools-extra/clang-query/tool/ClangQuery.cpp (diff)
The file was modifiedclang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp (diff)
The file was addedclang-tools-extra/test/clang-tidy/infrastructure/invalid-command-line.cpp
The file was addedclang-tools-extra/test/clang-query/invalid-command-line.cpp
Commit dfbfdc96f9e15be40c938cde9b159afd028bf4a2 by spatel
[utils] update expected strings in tests; NFC

The script was changes with:
https://github.com/llvm/llvm-project/commit/bfdc2552664d6f0bb332a9c6a115877020f3c1df
The file was modifiedclang/test/utils/update_cc_test_checks/Inputs/mangled_names.c.funcsig.expected (diff)
The file was modifiedclang/test/utils/update_cc_test_checks/Inputs/mangled_names.c.expected (diff)
Commit f4b0ebb89b3086a2bdd8c7dd1f5d142fa09ca728 by n.james93
Revert "clang-tidy and clang-query wont crash with invalid command line options"

This reverts commit f23ddbe3c3ae5f40b99ba272afc3d16b800ba8b9.
The file was removedclang-tools-extra/test/clang-tidy/infrastructure/invalid-command-line.cpp
The file was modifiedclang-tools-extra/clang-query/tool/ClangQuery.cpp (diff)
The file was modifiedclang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp (diff)
The file was removedclang-tools-extra/test/clang-query/invalid-command-line.cpp