[PATCH] D58354: PowerPC: Add ImmArg to intrinsics

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 12 04:04:05 PDT 2019


nemanjai accepted this revision.
nemanjai added a comment.
This revision is now accepted and ready to land.

Sorry about the delay. LGTM.

I just have a question about the implications of setting these flags - is this only to produce better errors (i.e. when an intrinsic call with a non-immediate argument appears in IR)? Or are there other implications as well?
I am just looking to gauge the importance of going through all the intrinsics to see which other ones also require this property.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58354/new/

https://reviews.llvm.org/D58354





More information about the llvm-commits mailing list