This is an archive of the discontinued LLVM Phabricator instance.

[Red Flag?] Handling invariant_start/end intrinsics.
Needs ReviewPublic

Authored by lvoufo on Sep 21 2015, 12:48 PM.



[Continued from D11827; sans 'writeonce' annotations.]

Almost done, modulo test cases and minor cleanups/generalizations.
I expect to have this ready to submit for review upstream tomorrow,
along with the updated design doc, sometime tomorrow (and definitely by the end of this week).

Please let me know if you see any red flag so far?

Diff Detail

Event Timeline

lvoufo updated this revision to Diff 35293.Sep 21 2015, 12:48 PM
lvoufo retitled this revision from to [Red Flag?] Handling invariant_start/end intrinsics..
lvoufo updated this object.
lvoufo added reviewers: chandlerc, majnemer, dberlin.
lvoufo updated this revision to Diff 35298.Sep 21 2015, 12:58 PM

Update diff with more visible context (-U999999).

lvoufo updated this revision to Diff 35776.Sep 25 2015, 3:48 PM

Add test cases...

lvoufo updated this revision to Diff 35809.Sep 26 2015, 12:53 PM

More test cases, and this is pretty much it.
Next Steps:

  • update design doc, and
  • submit this for revision upstream.
lvoufo added a comment.Oct 9 2015, 2:13 PM

This has been submitted as D13606.