[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