[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