[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