SuccessChanges

Summary

  1. [Support] Add more context to DataExtractor getLEB128 errors (details)
  2. [CodeGen] remove instnamer dependency from test file; NFC (details)
  3. [StructurizeCFG] Fix an incorrect comment, NFC. (details)
  4. Support GCC [[gnu::attributes]] in C2x mode (details)
  5. [openmp] Fixed taskloop recursive splitting so that taskloop tasks have (details)
  6. [InstNamer] use 'i' for Instructions, not 'tmp' (details)
Commit 8d9070e040d0aa916b3b63c319eabdf3e4a5f9df by james.henderson
[Support] Add more context to DataExtractor getLEB128 errors

Reviewed by: clayborg, dblaikie, labath

Differential Revision: https://reviews.llvm.org/D80799
The file was modifiedllvm/unittests/Support/DataExtractorTest.cpp
The file was modifiedllvm/test/tools/llvm-dwarfdump/X86/debug_line_short_prologue.s
The file was modifiedllvm/test/DebugInfo/X86/dwarfdump-debug-loclists-error-cases.s
The file was modifiedllvm/lib/Support/DataExtractor.cpp
Commit c0303e5391f65dbad3a6f1dbfa5ac9c9a83fa6c0 by spatel
[CodeGen] remove instnamer dependency from test file; NFC

This file was originally added without instnamer at:
rL283716 / fe2b9b4fbf860e3dc7da7705f548bc8d7b6ab9c1

But that was reverted and the test file reappeared with instnamer at:
rL285688 / 62f516f5906f967179610a73e4cc1d852b908bbd

I'm not seeing any difference locally from checking nameless values,
so trying to remove a layering violation and see if that can
survive the build bots.
The file was modifiedclang/test/CodeGen/x86-inline-asm-v-constraint.c
Commit 8a84158e5b966236c0e090cd5c7f44dab69e59a8 by ehudkatz
[StructurizeCFG] Fix an incorrect comment, NFC.
The file was modifiedllvm/lib/Transforms/Scalar/StructurizeCFG.cpp
Commit 522934da1f0c78c1de1a80d4ba14204a11f5afa8 by aaron
Support GCC [[gnu::attributes]] in C2x mode

GCC 10.1 introduced support for the [[]] style spelling of attributes in C
mode. Similar to how GCC supports __attribute__((foo)) as [[gnu::foo]] in
C++ mode, it now supports the same spelling in C mode as well. This patch
makes a change in Clang so that when you use the GCC attribute spelling,
the attribute is automatically available in all three spellings by default.
However, like Clang, GCC has some attributes it only recognizes in C++ mode
(specifically, abi_tag and init_priority), which this patch also honors.
The file was modifiedclang/test/Sema/attr-c2x.c
The file was modifiedclang/utils/TableGen/ClangAttrEmitter.cpp
The file was modifiedclang/include/clang/Basic/Attr.td
Commit 5e111c5df8efde39c62d5e6906f590311782e30b by Andrey.Churbanov
[openmp] Fixed taskloop recursive splitting so that taskloop tasks have
same parent tasks.

Differential Revision: https://reviews.llvm.org/D80577
The file was modifiedopenmp/runtime/src/kmp_tasking.cpp
The file was addedopenmp/runtime/test/tasking/omp_taskloop_taskwait.c
Commit dd54432a0f5a6f042fa4d2db3094c6f02e5ad275 by spatel
[InstNamer] use 'i' for Instructions, not 'tmp'

As discussed in https://bugs.llvm.org/show_bug.cgi?id=45951 and
D80584, the name 'tmp' is almost always a bad choice, but we have
a legacy of regression tests with that name because it was baked
into utils/update_test_checks.py.

This change makes -instnamer more consistent (already using "arg"
and "bb", the common LLVM shorthand). And it avoids the conflict
in telling users of the FileCheck script to run "-instnamer" to
create a better regression test and having that cause a warn/fail
in update_test_checks.py.
The file was modifiedllvm/lib/Transforms/Utils/InstructionNamer.cpp
The file was modifiedllvm/test/Transforms/InstNamer/basic.ll