[PATCH] D20116: Add speculatable function attribute

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 1 17:18:07 PST 2017

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

LGTM. But let's wait a little to give a change to @sanjoy or @chandlerc to comment if they feel the need.

I repeat here one of the important earlier comment from @tstellarAMD , since it is not in the description and easy to miss:

  I added two new Intrinsic attributes IntrNoSideEffects and IntrHasSideEffects,
  which make it possible to specify all the possible memory interaction / side effect
  combinations. With these properties in place, it should be possible in the future
  to drop the 'no side effect' portion of the intrinsic memory properties once targets
  have been updated to use these new properties.


More information about the llvm-commits mailing list