Page MenuHomePhabricator

[Alignment][NFC] Transitionning more getMachineMemOperand call sites

Authored by gchatelet on Mar 31 2020, 1:05 AM.



This is patch is part of a series to introduce an Alignment type.
See this thread for context:
See this patch for the introduction of the type:

Diff Detail

Event Timeline

gchatelet created this revision.Mar 31 2020, 1:05 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 31 2020, 1:05 AM
courbet accepted this revision.Mar 31 2020, 1:15 AM
courbet added inline comments.

This is a potential functional change, so I'll mention what I checked for posterity:
There are three instantiations of this class. In all cases, Alignment is never read before being written to.

This revision is now accepted and ready to land.Mar 31 2020, 1:15 AM
gchatelet marked 2 inline comments as done.Mar 31 2020, 1:35 AM
gchatelet added inline comments.

Yes I checked as well manually (by reading the code) and by running all tests and asserting when Alignment was read and equal to zero.

This revision was automatically updated to reflect the committed changes.
gchatelet marked an inline comment as done.