SuccessChanges

Changes from Git (git http://labmaster3.local/git/llvm-project.git)

Summary

  1. TargetLoweringObjectFile.h - remove unnecessary includes. NFCI. (details)
  2. TargetIntrinsicInfo.h - remove unnecessary Compiler.h include. NFC. (details)
  3. [AST] Fix recovery-AST crash: dependent overloaded call exprs are now possible. (details)
  4. [Tooling] Drop leading/trailing whitespace from compile_flags.txt lines (details)
  5. [clangd] Fix gcc compiler warning by removing extra ";" [NFC] (details)
  6. [yaml2obj] - Stop using square brackets for unique suffixes. (details)
  7. [Analyzer][VLASizeChecker] Try to fix vla.c test problems. (details)
Commit cdafe59f9528567df2e06c9f92f089791e3b9d5b by llvm-dev
TargetLoweringObjectFile.h - remove unnecessary includes. NFCI.

Replace with forward declarations and move includes down to source files where required.

I also needed to move the TargetLoweringObjectFile::SectionForGlobal wrapper implementation down into TargetLoweringObjectFile.cpp
The file was modifiedllvm/include/llvm/Target/TargetLoweringObjectFile.h
The file was modifiedllvm/lib/CodeGen/GlobalMerge.cpp
The file was modifiedllvm/lib/Target/Hexagon/HexagonTargetObjectFile.cpp
The file was modifiedllvm/lib/Target/Hexagon/HexagonTargetObjectFile.h
The file was modifiedllvm/lib/Target/TargetLoweringObjectFile.cpp
The file was modifiedllvm/lib/Target/PowerPC/PPCTargetObjectFile.cpp
Commit e215cb7aa48e811e325587d437022fa4b947c4a4 by llvm-dev
TargetIntrinsicInfo.h - remove unnecessary Compiler.h include. NFC.

We don't need use compiler specific attributes so don't need Compiler.h
The file was modifiedllvm/include/llvm/Target/TargetIntrinsicInfo.h
Commit 1d579f54d720dcc53e11386fdec59e07614599a5 by sam.mccall
[AST] Fix recovery-AST crash: dependent overloaded call exprs are now possible.

Reviewers: hokein

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D80154
The file was modifiedclang/lib/AST/StmtProfile.cpp
The file was modifiedclang/test/AST/ast-dump-recovery.cpp
Commit 48aa781ae44a611967084ca031fdbb3a0712c40c by sam.mccall
[Tooling] Drop leading/trailing whitespace from compile_flags.txt lines

Summary:
These files tend to be hand-authored, and people get very confused.
I can't think of any reason that such whitespace would be intended.

Reviewers: kadircet

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D80160
The file was modifiedclang/test/Tooling/fixed-database.cpp
The file was modifiedclang/lib/Tooling/CompilationDatabase.cpp
Commit e408b54290bd5dc35b02ef4270292d52c89d9751 by mikael.holmen
[clangd] Fix gcc compiler warning by removing extra ";" [NFC]

gcc complained with:

/data/repo/master/clang-tools-extra/clangd/index/Ref.h:189:2: warning: extra ';' [-Wpedantic]
}; // namespace llvm
  ^
The file was modifiedclang-tools-extra/clangd/index/Ref.h
Commit e2b134b01a6b1a201015c69ef83fe1861dd1139b by grimar
[yaml2obj] - Stop using square brackets for unique suffixes.

For describing section/symbol names we can use unique suffixes,
e.g:

```
- Name: '.foo [1]`
- Name: '.foo [2]`
```

It can be a problem (see https://reviews.llvm.org/D79984#inline-734829),
because `[]` are sometimes used to describe a macros:

```
- Name: "[[a0]]"
```

Seems the better approach is to use something else, like "()".
This patch does it and refactors the code related.

Differential revision: https://reviews.llvm.org/D80123
The file was modifiedllvm/test/tools/yaml2obj/ELF/stack-sizes.yaml
The file was modifiedllvm/test/tools/llvm-readobj/ELF/stack-sizes.test
The file was modifiedllvm/test/tools/llvm-gsymutil/X86/elf-dwarf.yaml
The file was modifiedllvm/test/tools/yaml2obj/ELF/duplicate-section-names.yaml
The file was modifiedllvm/test/tools/yaml2obj/ELF/call-graph-profile-section.yaml
The file was modifiedllvm/test/tools/obj2yaml/ELF/program-headers.yaml
The file was modifiedllvm/test/tools/obj2yaml/ELF/duplicate-symbol-and-section-names.yaml
The file was modifiedllvm/include/llvm/ObjectYAML/ELFYAML.h
The file was modifiedllvm/lib/ObjectYAML/ELFEmitter.cpp
The file was modifiedllvm/test/tools/llvm-objcopy/ELF/strip-dwo-groups.test
The file was modifiedllvm/test/tools/yaml2obj/ELF/duplicate-symbol-names.yaml
The file was modifiedllvm/test/tools/llvm-dwarfdump/X86/section_sizes_elf.test
The file was modifiedllvm/test/Object/X86/obj2yaml-dup-section-name.s
The file was modifiedllvm/tools/obj2yaml/elf2yaml.cpp
Commit 56079e1de1129837aa7569d8b3bb5e50afc0f1ea by 1.int32
[Analyzer][VLASizeChecker] Try to fix vla.c test problems.
The file was modifiedclang/test/Analysis/vla.c