[PATCH] D50931: Consider isLegalAddressingImm in Constant Hoisting
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 20 15:38:40 PDT 2018
efriedma edited reviewers, added: wmi, SjoerdMeijer, mehdi_amini, t.p.northover; removed: srhines, ributzka.
efriedma added a comment.
Please upload patches with full context (http://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface).
This could probably use a few more tests to show the correct type is getting passed to isLegalAddressingMode (e.g. on Thumb1, an i8 load only supports offsets 0-31, but an i32 load supports offsets 0-124.)
Repository:
rL LLVM
https://reviews.llvm.org/D50931
More information about the llvm-commits
mailing list