This is an archive of the discontinued LLVM Phabricator instance.

[ObjCARC] Use objc_msgSend instead of llvm.objc.msgSend in tests
ClosedPublic

Authored by nikic on Jul 15 2021, 1:10 PM.

Details

Summary

D55348 replaces @objc_msgSend with @llvm.objc.msgSend in tests together with many other objc intrinsics. However, this is not a recognized intrinsic (https://llvm.org/docs/LangRef.html#objective-c-arc-runtime-intrinsics) and does not receive special treatment by LLVM. It's likely that uses of this function were renamed by accident.

This came up in D106013, because the address of @llvm.objs.msgSend is taken, something which is normally not legal for intrinsics.

Diff Detail

Event Timeline

nikic requested review of this revision.Jul 15 2021, 1:10 PM
nikic created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJul 15 2021, 1:10 PM
This revision is now accepted and ready to land.Jul 15 2021, 1:17 PM
This revision was landed with ongoing or failed builds.Jul 15 2021, 1:22 PM
This revision was automatically updated to reflect the committed changes.