This is an archive of the discontinued LLVM Phabricator instance.

Fix comment about Intr*Mem properties
ClosedPublic

Authored by nhaehnle on Apr 22 2016, 9:33 AM.

Details

Summary

Follow up to D19291: it now makes sense to use two Intr*Mem properties,
in particular IntrReadMem + IntrArgMemOnly is common.

Pointed out by Mikael Holmén.

Diff Detail

Repository
rL LLVM

Event Timeline

nhaehnle updated this revision to Diff 54670.Apr 22 2016, 9:33 AM
nhaehnle retitled this revision from to Fix comment about Intr*Mem properties.
nhaehnle updated this object.
nhaehnle added reviewers: uabelho, mehdi_amini, reames.
nhaehnle added a subscriber: llvm-commits.
mehdi_amini accepted this revision.Apr 22 2016, 9:51 AM
mehdi_amini edited edge metadata.
This revision is now accepted and ready to land.Apr 22 2016, 9:51 AM
reames accepted this revision.Apr 22 2016, 10:29 AM
reames edited edge metadata.

LGTM. FYI, comment cleanup can be done without review.

This revision was automatically updated to reflect the committed changes.