SuccessChanges

Summary

  1. Revert "PR44685: DebugInfo: Handle address-use-invalid type units referencing non-type units" (details)
  2. [X86][AVX] lowerShuffleWithVTRUNC - avoid unnecessary division in element counts. NFCI. (details)
  3. [ELF] Hide target specific methods as private (details)
  4. [SyntaxTree] Implement annotation-based test infrastructure (details)
  5. [SyntaxTree] Use Annotations based tests for expressions (details)
  6. [OPENMP]Do not capture base pointer by reference if it is used as a base for array-like reduction. (details)
  7. [yaml2obj] - Don't crash when `FileHeader` declares an empty `Flags` key in specific situations. (details)
Commit b4bffdbadfcceb3959aaf231c1542301944e5812 by thakis
Revert "PR44685: DebugInfo: Handle address-use-invalid type units referencing non-type units"

This reverts commit be3ef93bf58aa5546c7baadfb21d43b75fbb4e24.
Test fails on macOS and Windows, e.g. http://45.33.8.238/win/22216/step_11.txt
The file was modifiedllvm/lib/CodeGen/AsmPrinter/DwarfDebug.h (diff)
The file was removedllvm/test/DebugInfo/X86/addr-tu-to-non-tu.ll
The file was modifiedllvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp (diff)
The file was modifiedllvm/lib/CodeGen/AsmPrinter/AddressPool.h (diff)
Commit 7db5124736d1b87e938f56c7eaa48f118fd328b0 by llvm-dev
[X86][AVX] lowerShuffleWithVTRUNC - avoid unnecessary division in element counts. NFCI.

(256 / SrcEltBits) == ((2 * EltSizeInBits * NumElts) / (EltSizeInBits * Scale)) == (2 * (NumElts / Scale)) == NumSrcElts
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.cpp (diff)
Commit 7b777ee73077dff4d8f3a0160f2dcf977895fd57 by RonakNilesh.Chauhan
[ELF] Hide target specific methods as private

Reviewed By: jhenderson

Differential Revision: https://reviews.llvm.org/D86136
The file was modifiedllvm/include/llvm/Object/ELFObjectFile.h (diff)
Commit ab58c9ee8a6e9ace3a93198496b4d85e8cb2b5a9 by ecaldas
[SyntaxTree] Implement annotation-based test infrastructure

We add the method `SyntaxTreeTest::treeDumpEqualOnAnnotations`, which
allows us to compare the treeDump of only annotated code. This will reduce a
lot of noise from our `BuildTreeTest` and make them short and easier to
read.
The file was modifiedclang/unittests/Tooling/Syntax/TreeTestBase.cpp (diff)
The file was modifiedclang/unittests/Tooling/Syntax/TreeTestBase.h (diff)
Commit c8c92b54d74c1b9256f9aed6ba89d66fbd1d01ae by ecaldas
[SyntaxTree] Use Annotations based tests for expressions

In this process we also create some other tests, in order to not lose
coverage when focusing on the annotated code

Differential Revision: https://reviews.llvm.org/D85962
The file was modifiedclang/unittests/Tooling/Syntax/BuildTreeTest.cpp (diff)
The file was modifiedclang/unittests/Tooling/Syntax/TreeTestBase.cpp (diff)
Commit 1b93ebccaa094c079db7ad729e2f7fea7bac2f34 by a.bataev
[OPENMP]Do not capture base pointer by reference if it is used as a base for array-like reduction.

If the declaration is used in the reduction clause, it is captured by
reference by default. But if the declaration is a pointer and it is a
base for array-like reduction, this declaration can be captured by
value, since the pointee is reduced but not the original declaration.

Differential Revision: https://reviews.llvm.org/D85321
The file was modifiedclang/test/OpenMP/parallel_for_reduction_task_codegen.cpp (diff)
The file was modifiedclang/test/OpenMP/parallel_sections_reduction_task_codegen.cpp (diff)
The file was modifiedclang/test/OpenMP/sections_reduction_task_codegen.cpp (diff)
The file was modifiedclang/lib/Sema/SemaOpenMP.cpp (diff)
The file was modifiedclang/test/OpenMP/target_parallel_reduction_task_codegen.cpp (diff)
The file was modifiedclang/test/OpenMP/distribute_parallel_for_reduction_task_codegen.cpp (diff)
The file was modifiedclang/test/OpenMP/for_reduction_task_codegen.cpp (diff)
The file was modifiedclang/test/OpenMP/parallel_reduction_task_codegen.cpp (diff)
The file was modifiedclang/test/OpenMP/target_teams_distribute_parallel_for_reduction_task_codegen.cpp (diff)
The file was modifiedclang/test/OpenMP/teams_distribute_parallel_for_reduction_task_codegen.cpp (diff)
The file was modifiedclang/test/OpenMP/parallel_master_reduction_task_codegen.cpp (diff)
The file was modifiedclang/test/OpenMP/target_parallel_for_reduction_task_codegen.cpp (diff)
Commit bd7daf5ceb92db00d3fc5d1ce8d4f74dcd03ebb9 by grimar
[yaml2obj] - Don't crash when `FileHeader` declares an empty `Flags` key in specific situations.

We currently call the `llvm_unreachable` for the following YAML:

```
--- !ELF
FileHeader:
  Class:   ELFCLASS32
  Data:    ELFDATA2LSB
  Type:    ET_REL
  Machine: EM_NONE
  Flags:   [ ]
```

it happens because the `Flags` key is present, though `EM_NONE` is a
machine type that has no known `EF_*` values and we call `llvm_unreachable` by mistake.

Differential revision: https://reviews.llvm.org/D86138
The file was addedllvm/test/tools/yaml2obj/ELF/eflags.yaml
The file was modifiedllvm/lib/ObjectYAML/ELFYAML.cpp (diff)