[PATCH] D153356: [Align] Add isAligned taking an APInt

[Nikita Popov via Phabricator via llvm-commits]

nikic added inline comments.


================
Comment at: llvm/lib/Support/APInt.cpp:168
 
+/// Prefix increment operator. Increments the APInt by one.
+bool APInt::isAligned(Align A) const {
----------------
Copy and paste mistake :) You can just drop the comment as it's already in the header.


================
Comment at: llvm/unittests/ADT/APIntTest.cpp:1861
+    bool isAligned;
+  } kTests[] = {
+      {1, 0, true},  {1, 1, true},  {1, 5, true},  {2, 0, true},
----------------
What does the `k` here mean?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153356