[PATCH] D122130: Verify parameter alignment attribute

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 21 10:21:59 PDT 2022


craig.topper added inline comments.
Herald added a subscriber: StephenFan.


================
Comment at: llvm/lib/IR/Verifier.cpp:1818
+        Align AttrAlign = Attrs.getAlignment().valueOrOne();
+        // ArgFlagsTy::MemAlign only have 4 bits for alignment, so the
+        // alignment size should not exceed 2^15. Since encode(Align)
----------------
ArgFlagsTy -> ISD::ArgFlagsTy


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122130



More information about the llvm-commits mailing list