[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