[PATCH] D59971: [GlobalISel] Add legalization support for non-power-2 loads and stores
Amara Emerson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 15 15:14:23 PDT 2019
aemerson reclaimed this revision.
aemerson added a comment.
Reviving this as the overall approach was fine, it seems the alignment of non pow2 types is assumed to be the alignment of the next largest pow-2 type, so we don't need to worry about alignment during the breakdown.
I did however change the legalization method to not use extracts/inserts, but instead use extending loads and truncating stores, so that the artifacts get combined away and it Just Works.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59971/new/
https://reviews.llvm.org/D59971
More information about the llvm-commits
mailing list