This is an archive of the discontinued LLVM Phabricator instance.

[NFC] Encapsulate MemOp logic
ClosedPublic

Authored by gchatelet on Feb 4 2020, 12:54 AM.

Details

Summary

This patch simply introduces functions instead of directly accessing the fields.
This helps introducing additional check logic. A second patch will add simplifying functions.

Diff Detail

Event Timeline

gchatelet created this revision.Feb 4 2020, 12:54 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 4 2020, 12:54 AM
gchatelet updated this revision to Diff 242270.Feb 4 2020, 1:06 AM
  • Remove asserts for now
courbet accepted this revision.Feb 4 2020, 1:12 AM
courbet added inline comments.
llvm/include/llvm/CodeGen/TargetLowering.h
115

This can no longer be zero, please update the comment.

124–125

ditto

This revision is now accepted and ready to land.Feb 4 2020, 1:12 AM
gchatelet updated this revision to Diff 242272.Feb 4 2020, 1:36 AM
  • Address comments
This revision was automatically updated to reflect the committed changes.