[PATCH] [ARM] Improve handling of stack accesses in Thumb-1.

Renato Golin renato.golin at linaro.org
Wed Feb 18 10:26:41 PST 2015


In http://reviews.llvm.org/D7594#125493, @john.brawn wrote:

> New patch with test failures fixed. Some test failures were due to the load/store optimizer not understanding the AddrModeT1_s loads/stores, so it now does understand them, but an annoyingly large amount were due to tests needing adjustment due to different codegen. The widening of loads also now only happens little-endian, and not in thumb2 as then we have other instructions with more flexible addressing.


Most of the tests were not generic enough to begin with. But due to their specific nature, I'm not sure we can improve them more than what you did.

James, are you happy with the LoadStoreOptimizer changes?

Otherwise, LGTM. Thanks!


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D7594

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list