[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