This is an archive of the discontinued LLVM Phabricator instance.

[mlir] Add alignment attribute to LLVM memory ops and use in vector.transfer
ClosedPublic

Authored by nicolasvasilache on Jul 13 2020, 8:04 AM.

Details

Summary

The native alignment may generally not be used when lowering a vector.transfer to the underlying load/store operation. This revision fixes the unmasked load/store alignment to match that of the masked path.

Diff Detail

Event Timeline

Herald added a reviewer: ftynse. · View Herald Transcript
Herald added a reviewer: aartbik. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
bkramer accepted this revision.Jul 13 2020, 8:06 AM

looks good!

This revision is now accepted and ready to land.Jul 13 2020, 8:06 AM
aartbik accepted this revision.Jul 13 2020, 9:06 AM
This revision was automatically updated to reflect the committed changes.