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

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

We have `isa<DbgInfoIntrinsic>` to check for all debug info.

More generally, I'm not sure there's a definition of "metadata" that would suit enough passes to make it worthwhile.  There's a function for estimating the code size of a call: getIntrinsicCost in TTI returns TCC_Free for a bunch of intrinsics.  llvm::isAssumeLikeIntrinsic checks... well, I'm actually not quite sure what it checks; it currently looks for metadata-ish intrinsics, but the documentation for it doesn't really match that meaning.  And we have isLoweredToCall, which you were looking into changing.  So I guess there's demand for something, but I'm not quite sure what it looks like.


https://reviews.llvm.org/D45216