SuccessChanges

Summary

  1. [clang-format] Respect spaces in line comment section... (details)
  2. [clang-format][PR47290] Add ShortNamespaceLines format option (details)
  3. [lld/mac] Implement the missing bits of -undefined (details)
  4. Revert "[c++20] Mark class type NTTPs as done and start defining the feature test macro." (details)
Commit 418b4a7b3158b47043c4c8891eb2e27966a55fa2 by bjoern
[clang-format] Respect spaces in line comment section...

... without an active column limit.

Before line comments were not touched at all with ColumnLimit == 0.

Differential Revision: https://reviews.llvm.org/D96896
The file was modifiedclang/unittests/Format/FormatTest.cpp (diff)
The file was modifiedclang/unittests/Format/FormatTestComments.cpp (diff)
The file was modifiedclang/lib/Format/ContinuationIndenter.cpp (diff)
Commit 6ca52815fb3cb32343b9a008fdf9f593da39f203 by bjoern
[clang-format][PR47290] Add ShortNamespaceLines format option

clang-format documentation states that having enabled
FixNamespaceComments one may expect below code:

c++
namespace a {
foo();
}

to be turned into:

c++
namespace a {
foo();
} // namespace a

In reality, no "// namespace a" was added. The problem was too high
value of kShortNamespaceMaxLines, which is used while deciding whether
a namespace is long enough to be formatted.

As with 9163fe2, clang-format idempotence is preserved.

Differential Revision: https://reviews.llvm.org/D87587
The file was modifiedclang/docs/ClangFormatStyleOptions.rst (diff)
The file was modifiedclang/docs/ReleaseNotes.rst (diff)
The file was modifiedclang/include/clang/Format/Format.h (diff)
The file was modifiedclang/lib/Format/NamespaceEndCommentsFixer.cpp (diff)
The file was modifiedclang/unittests/Format/NamespaceEndCommentsFixerTest.cpp (diff)
The file was modifiedclang/lib/Format/Format.cpp (diff)
Commit 0658fc654c758d0bfab7d119ecf94caf5e5276d5 by thakis
[lld/mac] Implement the missing bits of -undefined

This adds support for `-undefined dynamic_lookup`, and for
`-undefined warning` and `-undefined suppress` with `-flat_namespace`.

We just replace undefined symbols with a DynamicLookup when we hit them.

With this, `check-llvm` passes when using ld64.lld.darwinnew as host linker.

Differential Revision: https://reviews.llvm.org/D97642
The file was modifiedlld/MachO/SymbolTable.cpp (diff)
The file was modifiedlld/MachO/Writer.cpp (diff)
The file was modifiedlld/MachO/UnwindInfoSection.cpp (diff)
The file was modifiedlld/test/MachO/treat-undef-sym.s (diff)
Commit 564f5b0734bd5d265a0046e5ca9d08ae5bc303eb by richard
Revert "[c++20] Mark class type NTTPs as done and start defining the feature test macro."

Some of the parts of this work were reverted; stop defining the feature
test macro for now.

This reverts commit b4c63ef6dd90dba9af26a111c9a78b121c5284b1.
The file was modifiedclang/lib/Frontend/InitPreprocessor.cpp (diff)
The file was modifiedclang/test/Lexer/cxx-features.cpp (diff)