[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.
More information about the llvm-commits