[PATCH] D106013: [Verifier] Require same signature for intrinsic calls
Arthur Eubanks via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 15 09:53:49 PDT 2021
aeubanks added a comment.
`isIntrinsic()` looks like it's equivalent to `getName().startswith("llvm.")`.
Is it expected that we can take the address of not-registered intrinsics? That seems wrong. I'd say we should disallow that, making it an easy decision to keep what we have and fix the test.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106013/new/
https://reviews.llvm.org/D106013
More information about the llvm-commits
mailing list