[PATCH] D16890: Fix PR25339: ARM Constant Island

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 21 04:34:17 PST 2016


rengolin added a comment.

In http://reviews.llvm.org/D16890#356160, @llvm-commits wrote:

> This patch doesn't reduce the threshold to 15. It tries to accelerate 
>  the converge after 15th iteration by ignoring the "best growth".


Ok, so then we need the number 15 to be on the same place as the number 30. If both are magic constants, we need to make them into some kind of static const global or a hidden compiler flag.

cheers,
--renato


Repository:
  rL LLVM

http://reviews.llvm.org/D16890





More information about the llvm-commits mailing list