[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