SuccessChanges

Summary

  1. [NFC, StackSafety] Change type of internal container (details)
  2. [StackSafety] Skip non-pointer parameters (details)
  3. [clang-tidy] Added MacroDefiniton docs for readability-identifier-naming (details)
  4. Revert "[WebAssembly] Eliminate range checks on br_tables" (details)
  5. [gn build] Port 755a8959152 (details)
  6. [AMDGPU/MemOpsCluster] Code clean-up around accessing of memory operand width (details)
  7. [AMDGPU] Fold llvm.amdgcn.cos and llvm.amdgcn.sin intrinsics (details)
  8. [StackSafety,NFC] Rename internal class (details)
  9. [StackSafety,NFC] Convert to template internal stuff (details)
  10. Revert "[Support] Add file lock/unlock functions" (details)
  11. [AMDGPU] Fold llvm.amdgcn.cos and llvm.amdgcn.sin intrinsics (fix) (details)
  12. Syntax tree: ignore implicit expressions at the top level of statements (details)
  13. Split syntax tree tests into more granular ones (details)
  14. [CSInfo][MIPS][DwarfDebug] Add support for delay slots (details)
Commit e128f01be996f5f9a0e0cf5565d8a0696783b85d by Vitaly Buka
[NFC, StackSafety] Change type of internal container

Summary: Depends on D80771.

Reviewers: eugenis

Reviewed By: eugenis

Subscribers: mehdi_amini, hiraditya, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D80847
The file was modifiedllvm/lib/Analysis/StackSafetyAnalysis.cpp
Commit d3b7f90d00fc23f5e5576e28fe4290b5f1dd0a9d by Vitaly Buka
[StackSafety] Skip non-pointer parameters

Summary: Depends on D80908.

Reviewers: eugenis, pcc

Reviewed By: eugenis

Subscribers: hiraditya, steven_wu, dexonsmith, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D80956
The file was modifiedllvm/test/Analysis/StackSafetyAnalysis/ipa.ll
The file was modifiedllvm/test/Analysis/StackSafetyAnalysis/memintrin.ll
The file was modifiedllvm/test/Analysis/StackSafetyAnalysis/local.ll
The file was modifiedllvm/lib/Analysis/StackSafetyAnalysis.cpp
Commit 65fa0a9f7f3e6090e335c14f9edea19358d87613 by n.james93
[clang-tidy] Added MacroDefiniton docs for readability-identifier-naming

Updates the docs to include `MacroDefinition` documentation. The docs are still missing `ObjCIVar` however I don't have a clue about how that looks in code. If someone wants to show the code block needed for the example I'll add that in too.

Reviewed By: aaron.ballman

Differential Revision: https://reviews.llvm.org/D80877
The file was modifiedclang-tools-extra/docs/clang-tidy/checks/readability-identifier-naming.rst
Commit 755a89591528b692315ad0325347e2fd4637271b by tlively
Revert "[WebAssembly] Eliminate range checks on br_tables"

This reverts commit f99d5f8c32a822580a732d15a34e8197da55d22b.
The change was causing UBSan and other failures on some bots.
The file was modifiedllvm/lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCTargetDesc.h
The file was removedllvm/lib/Target/WebAssembly/WebAssemblyFixBrTableDefaults.cpp
The file was modifiedllvm/test/CodeGen/WebAssembly/switch.ll
The file was modifiedllvm/test/CodeGen/WebAssembly/cfg-stackify.ll
The file was modifiedllvm/test/CodeGen/WebAssembly/indirectbr.ll
The file was modifiedllvm/lib/Target/WebAssembly/CMakeLists.txt
The file was modifiedllvm/lib/Target/WebAssembly/WebAssembly.h
The file was modifiedllvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp
The file was removedllvm/test/CodeGen/WebAssembly/switch-unreachable-default.ll
The file was modifiedllvm/lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp
The file was modifiedllvm/test/CodeGen/WebAssembly/stack-insts.ll
Commit f2f3a80fd853fba0c7bd763a92c0d445052693a8 by llvmgnsyncbot
[gn build] Port 755a8959152
The file was modifiedllvm/utils/gn/secondary/llvm/lib/Target/WebAssembly/BUILD.gn
Commit 29c17ed96ed53399279c3358cffde51224178e72 by mahesha.comp
[AMDGPU/MemOpsCluster] Code clean-up around accessing of memory operand width

Summary:
Clean-up the width computing logic given a memory operand, and re-arrange code to avoid
code duplication.

Reviewers: foad, rampitec, arsenm, vpykhtin, javedabsar

Reviewed By: foad

Subscribers: kzhuravl, jvesely, wdng, nhaehnle, yaxunl, dstuttard, tpr, t-tye, hiraditya, kerbowa, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D80946
The file was modifiedllvm/lib/Target/AMDGPU/SIInstrInfo.h
The file was modifiedllvm/lib/Target/AMDGPU/SIInstrInfo.cpp
Commit c823cfde21b20c60f8cf02e28af234e01964469a by jay.foad
[AMDGPU] Fold llvm.amdgcn.cos and llvm.amdgcn.sin intrinsics

Differential Revision: https://reviews.llvm.org/D80702
The file was addedllvm/test/Analysis/ConstantFolding/AMDGPU/sin.ll
The file was modifiedllvm/lib/Analysis/ConstantFolding.cpp
The file was addedllvm/test/Analysis/ConstantFolding/AMDGPU/cos.ll
Commit a019579fe56819688a36ba718afc8c7c4f3b04b1 by Vitaly Buka
[StackSafety,NFC] Rename internal class
The file was modifiedllvm/lib/Analysis/StackSafetyAnalysis.cpp
Commit 6e51a080f744013eaaff6be2f4864afd0882acfa by Vitaly Buka
[StackSafety,NFC] Convert to template internal stuff

It's going to be usefull for ThinLTO.
The file was modifiedllvm/lib/Analysis/StackSafetyAnalysis.cpp
Commit 8577595e03faf740ee0cfae1bbb2d0ff6f4e4516 by sepavloff
Revert "[Support] Add file lock/unlock functions"

This reverts commit f51bc4fb60fbcef26d18eff549fc68307fd46489.
It broke the Solaris buildbots (Builder clang-solaris11-sparcv9 Build #5494
<http://lab.llvm.org:8014/builders/clang-solaris11-sparcv9/builds/54).
The file was modifiedllvm/include/llvm/Support/FileSystem.h
The file was modifiedllvm/lib/Support/Windows/Path.inc
The file was modifiedllvm/unittests/Support/Path.cpp
The file was modifiedllvm/lib/Support/Unix/Path.inc
Commit 7c7941fb4bd884ee7dce96fdfdf657c0fdb608dd by jay.foad
[AMDGPU] Fold llvm.amdgcn.cos and llvm.amdgcn.sin intrinsics (fix)

Try to fix Windows buildbots.
The file was modifiedllvm/lib/Analysis/ConstantFolding.cpp
Commit b34b7691facd89022e7fee174debdbd2bf7920f3 by gribozavr
Syntax tree: ignore implicit expressions at the top level of statements

Summary:
I changed `markStmtChild` to ignore implicit expressions the same way as
`markExprChild` does it already. The test that I modified crashes
without this change.

Reviewers: hlopko, eduucaldas

Reviewed By: hlopko, eduucaldas

Subscribers: gribozavr2, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D81019
The file was modifiedclang/lib/Tooling/Syntax/BuildTree.cpp
The file was modifiedclang/unittests/Tooling/Syntax/TreeTest.cpp
Commit d7d5dd31fc6f05daf4758e1523c86401aa4e3f2a by gribozavr
Split syntax tree tests into more granular ones

Summary:
Doing so allows us to increase test coverage by removing unnecessary
language restrictions.

Reviewers: hlopko, eduucaldas

Reviewed By: hlopko, eduucaldas

Subscribers: gribozavr2, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D81040
The file was modifiedclang/unittests/Tooling/Syntax/TreeTest.cpp
Commit dd1bc59b729a31d6c0fe3aecc8559b9349363bf9 by djordje.todorovic
[CSInfo][MIPS][DwarfDebug] Add support for delay slots

This adds call site info support for call instructions with delay slot.
Search for instructions inside call delay slot, which load value
into parameter forwarding registers.
Return address of the call points to instruction after call delay slot,
which is not the one, immediately after the call instruction.

Patch by Nikola Tesic

Differential revision: https://reviews.llvm.org/D78107
The file was addedllvm/test/DebugInfo/Mips/dbg-call-site-low-pc.ll
The file was addedllvm/test/DebugInfo/MIR/Mips/dbg-call-site-delay-slot-interpretation.mir
The file was addedllvm/test/DebugInfo/MIR/Mips/dbg-call-site-delay-slot-interpretation-64bit.mir
The file was modifiedllvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp