[PATCH] D41104: Set the NoRecurse attribute for the dbg intrinsics.

Matt Davis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 15 16:44:51 PST 2018


mattd added a comment.

In https://reviews.llvm.org/D41104#1009477, @efriedma wrote:

> Not sure it needs any refactoring?  I'm more concerned about the "if (F->isIntrinsic()) return false;" part.


Thanks for the response.  As you mentioned earlier in this post, what we really need is an attribute or IntrinsicProperty that specifies whether a particular intrinsic becomes a call instruction.   It seems the simplest approach, then, would be to add a property that states if an intrinsic is truly a call.  If the property is not set, then the intrinsic is not a call, which is probably true for  most of the intrinsics.


https://reviews.llvm.org/D41104





More information about the llvm-commits mailing list