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

[Matt Davis via Phabricator via llvm-commits]

mattd updated this revision to Diff 129464.
mattd added a comment.

Thanks for the suggestion @hfinkel, and I agree, using the isLoweredToCall predicate seems the most comprehensive solution; however, I did find it a bit more intrusive than I was expecting.  As a side-note, I slapped together a much simpler/less-intrusive patch, that only special-cased the dbg intrinsics and not checking the lowering information, but I feel using the lowering data is more correct.


https://reviews.llvm.org/D41104

Files:
  lib/Transforms/IPO/FunctionAttrs.cpp
  test/Analysis/TypeBasedAliasAnalysis/functionattrs.ll
  test/CodeGen/Generic/dbg-influenced-attrs.ll
  test/Other/cgscc-devirt-iteration.ll
  test/Other/new-pm-lto-defaults.ll
  test/Transforms/FunctionAttrs/int_sideeffect.ll
  test/Transforms/FunctionAttrs/norecurse.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41104.129464.patch
Type: text/x-patch
Size: 11506 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180111/82bc1bcc/attachment.bin>