[PATCH] D45216: [Attributes] Add IntrinsicLoweredToCall attribute.

[Matt Davis via Phabricator via llvm-commits]

mattd added a comment.

Thanks for the feedback @eli.friedman.

Aside from the norecurse cases, there are other places that could probably benefit from knowing if a particular intrinsic is either a call to some user code, just an instruction, or metadata.  As you mentioned, this is target specific.   I think it makes sense to make isLoweredToCall a virtual function in TargetLowering. Each target-specific implementation of TargetLowering could override that routine to handle specific instructions that may/may-not be lowered to a user call. As an initial pass we could take the routine as it is now and make that the default behavior, which doesn't change functionality, but opens the doors for other targets to make use of this functionality later.


https://reviews.llvm.org/D45216