Page MenuHomePhabricator

[LLVM][Alignment] Convert isLegalNTStore/isLegalNTLoad to llvm::Align
ClosedPublic

Authored by gchatelet on Thu, Sep 5, 5:32 AM.

Details

Diff Detail

Repository
rL LLVM

Event Timeline

gchatelet created this revision.Thu, Sep 5, 5:32 AM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Sep 5, 5:32 AM
courbet added inline comments.Thu, Sep 5, 5:42 AM
llvm/lib/Transforms/Vectorize/LoopVectorizationLegality.cpp
741 ↗(On Diff #218898)

Given that getLoadStoreAlignment is eventually going to return a MaybeAlign, I would assert here.

gchatelet updated this revision to Diff 218901.Thu, Sep 5, 5:54 AM
gchatelet marked an inline comment as done.
  • Assert if alignment is not set
courbet accepted this revision.Thu, Sep 5, 5:59 AM
This revision is now accepted and ready to land.Thu, Sep 5, 5:59 AM
This revision was automatically updated to reflect the committed changes.