[PATCH] D124705: [ADT][NFC] Add missing closing comment for `llvm` namespace
Whisperity via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Apr 30 05:12:29 PDT 2022
whisperity added a comment.
This is unrelated to Clang-Tidy, the change affects the LLVM ADT library.
(Also, what is the policy for commits like these? This seems to be like borderline noise, a simple formatting commit, nothing else.)
It is not achieving anything, but also breaks the formatting rules that are found in other parts of the project. For example, `APInt.h` is terminated with `} // namespace llvm` and not `} // `**`end`**` namespace llvm`, see http://github.com/llvm/llvm-project/blob/40a2e35599b5e5fa0dd93ac4edf4bf02ee636f6e/llvm/include/llvm/ADT/APInt.h#L2301.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D124705/new/
https://reviews.llvm.org/D124705
More information about the llvm-commits
mailing list