[PATCH] D106013: [Verifier] Require same signature for intrinsic calls
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 15 12:13:59 PDT 2021
nikic added a comment.
In D106013#2881094 <https://reviews.llvm.org/D106013#2881094>, @dexonsmith wrote:
> In D106013#2881021 <https://reviews.llvm.org/D106013#2881021>, @aeubanks wrote:
>
>> @ahatanak, any idea what's up with `@llvm.objc.msgSend()`?
>
> There's some context in https://reviews.llvm.org/D55348 / be4f5711073613115c036206db0d9a45fd0632ab.
Seeing that, I wonder if this was some overzealous search and replace in tests? At least the ModuleHasARC() function doesn't contain `llvm.objc.msgSend`. And the other intrinsics introduced there are emitted by clang (or at least some I checked), but this one is not.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106013/new/
https://reviews.llvm.org/D106013
More information about the llvm-commits
mailing list