[PATCH] D69211: AMDGPU: Enable shouldConsiderGEPOffsetSplit

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 19 00:11:18 PDT 2019


rampitec added a comment.

The RP blow up doesn't look good. This is probably our biggest problem, code size does not compensate for it. Given the purpose of the test to be able to fit into a minimal register footprint, I think it will lead to quite bad regressions. We need something smarter than that.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69211/new/

https://reviews.llvm.org/D69211





More information about the llvm-commits mailing list