SuccessChanges

Summary

  1. [clangd] Add main file macros into the main-file index. (details)
  2. Change XCore code owner. (details)
  3. [clang][cli] NFC: Remove SSPBufferSize assignment (details)
  4. [clang][cli] Port more CodeGenOptions to marshalling infrastructure (details)
  5. Revert "Fix llvm::Optional build breaks in MSVC using std::is_trivially_copyable" (details)
  6. [clang-tidy] Use DenseSet<SourceLocation> in UpgradeDurationConversionsCheck, NFCI (details)
  7. [flang][driver] Unify f18_version.h.in and Version.inc.in (details)
  8. [InferFunctionAttrs] Improve CHECK variable names (NFC). (details)
Commit 2e25be0b6134e9544f7cee7bb7b31a921ca37cc0 by platonov.aleksandr
[clangd] Add main file macros into the main-file index.

This patch is a try to fix `WorkspaceSymbols.Macros` test after D93796.
If a macro definition is in the preamble section, then it appears to be in the preamble (static) index and not in the main-file (dynamic) index.
Thus, a such macro could not be found at a symbol search according to the logic that we skip symbols from the static index if the location of these symbols is inside the dynamic index files.
To fix this behavior this patch adds main file macros into the main-file (dynamic) index.

Reviewed By: sammccall

Differential Revision: https://reviews.llvm.org/D94477
The file was modifiedclang-tools-extra/clangd/XRefs.cpp
The file was modifiedclang-tools-extra/clangd/CollectMacros.h
The file was modifiedclang-tools-extra/clangd/unittests/ParsedASTTests.cpp
The file was modifiedclang-tools-extra/clangd/SemanticHighlighting.cpp
The file was modifiedclang-tools-extra/clangd/unittests/CollectMacrosTests.cpp
The file was modifiedclang-tools-extra/clangd/unittests/FindSymbolsTests.cpp
The file was modifiedclang-tools-extra/clangd/CollectMacros.cpp
The file was modifiedclang-tools-extra/clangd/index/SymbolCollector.cpp
Commit 19297a4171177b942832d49a3767609d2a112ae1 by nigelp
Change XCore code owner.

Discussion: https://lists.llvm.org/pipermail/llvm-dev/2021-January/147603.html
The file was modifiedllvm/CODE_OWNERS.TXT
Commit 3bccd87a588b3c320b669686c8f006b92ff72182 by jan_svoboda
[clang][cli] NFC: Remove SSPBufferSize assignment

This should've been part of D84669, but got overlooked. Removing the assignment is NFC, as it's also done by the marshalling infrastructure for the stack_protector_buffer_size option.

Reviewed By: dexonsmith in D94488
The file was modifiedclang/lib/Frontend/CompilerInvocation.cpp
Commit fa2fe9608c1c1b402296960b1edc157230c30062 by jan_svoboda
[clang][cli] Port more CodeGenOptions to marshalling infrastructure

Leveraging the recently added TableGen constructs (ShouldParseIf and MarshallingInfoStringInt) to shift from manual command line parsing to automatic TableGen-driver marshalling.

Reviewed By: dexonsmith

Differential Revision: https://reviews.llvm.org/D94488
The file was modifiedclang/lib/Frontend/CompilerInvocation.cpp
The file was modifiedclang/include/clang/Driver/Options.td
Commit 6abbba3fca9fdf8d31f74800a7ddb40b103ae6e3 by alexandre.ganea
Revert "Fix llvm::Optional build breaks in MSVC using std::is_trivially_copyable"

This reverts commit 854f0984f0b7ab9a9a541a4bcda7ea173e4113d3.

This breaks compilation with clang-cl on Windows, while in a MSVC 16.8 cmd.exe.
This also breaks PPC: http://lab.llvm.org:8011/#/builders/93/builds/1435
And: https://reviews.llvm.org/D93510#2497737
The file was modifiedllvm/include/llvm/ADT/Optional.h
The file was modifiedllvm/unittests/ADT/OptionalTest.cpp
Commit 176f5e95e1afad75ff045a00f0fa9c781bd5f54a by mikhail.maltsev
[clang-tidy] Use DenseSet<SourceLocation> in UpgradeDurationConversionsCheck, NFCI

This change replaces `unordered_set<unsigned>` (which used to store
internal representation of `SourceLocation`-s) with
`DenseSet<SourceLocation>` (which stores `SourceLocation`-s directly).

Reviewed By: aaron.ballman, njames93

Differential Revision: https://reviews.llvm.org/D94601
The file was modifiedclang-tools-extra/clang-tidy/abseil/UpgradeDurationConversionsCheck.cpp
The file was modifiedclang-tools-extra/clang-tidy/abseil/UpgradeDurationConversionsCheck.h
Commit a1bbd1ef1dbd2a39777ef0b2d623f15f0523cdbf by andrzej.warzynski
[flang][driver] Unify f18_version.h.in and Version.inc.in

Flang has two CMake configurable header files that define compiler
version numbers:
* f18_version.h.in - only used in f18.cpp (uses version numbers from
  LLVM's macro definitions)
* Version.inc.in - not currently used (uses version numbers hard-coded
  in Flang's top CMake script)

Currently only f18_version.h.in provides version numbers consistent with
other subprojects in llvm-project. However, its location and name are
inconsistent with e.g. Clang. This patch merges the two headers
together:
  * hard-coded version numbers in Flang's top CMake script are deleted
  * Version.inc.in is updated to provide string versions of version
  numbers (required by f18.cpp)
  * f18_version.h.in is deleted as it's no longer needed

Differential Revision: https://reviews.llvm.org/D94422
The file was modifiedflang/include/flang/Version.inc.in
The file was removedflang/tools/f18/f18_version.h.in
The file was modifiedflang/tools/f18/CMakeLists.txt
The file was modifiedflang/CMakeLists.txt
The file was modifiedflang/tools/f18/f18.cpp
Commit c23e34e606bf23ee8d4060469ecebe0f7971c5dd by flo
[InferFunctionAttrs] Improve CHECK variable names (NFC).
The file was modifiedllvm/test/Transforms/InferFunctionAttrs/annotate.ll