[PATCH] D64790: [LLVM][NFC] Adding an Alignment type to LLVM

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 31 12:46:24 PDT 2019


aprantl added inline comments.


================
Comment at: llvm/trunk/include/llvm/Support/Alignment.h:39
+// alignment.
+// It is suitable for use as static global constants.
+struct Align {
----------------
Can you please doxygen-ify all the comments in this file? I.e. `//` -> `///`


================
Comment at: llvm/trunk/include/llvm/Support/Alignment.h:113
+// isAligned: Checks that SizeInBytes is a multiple of the alignment.
+// -----------------------------------------------------------------------------
+
----------------
This comment also doesn't follow the LLVM rules. It should be just
`/// Checks that ...`


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64790/new/

https://reviews.llvm.org/D64790





More information about the llvm-commits mailing list