[PATCH] D136330: [Verifier] Relieve intrinsics parameter alignment size constrain

LuoYuanke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 20 19:06:26 PDT 2022


LuoYuanke added inline comments.


================
Comment at: llvm/lib/IR/Verifier.cpp:3228-3230
+  if (IsIntrinsic)
+    Check(Callee->getValueType() == FTy,
+          "Intrinsic called with incompatible signature", Call);
----------------
skan wrote:
> Why do we need this check? Is there a LIT test for it?
This is the original code, and I just move it above `VerifyTypeAlign` check. See the left line 3240 ~ 3245.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136330



More information about the llvm-commits mailing list