[PATCH] D21183: Better selection of common base address in constant hoisting

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 10 08:55:47 PDT 2016


mcrosier added a comment.

In http://reviews.llvm.org/D21183#453606, @ributzka wrote:

> Constants are not only used by load/store instructions, so using "isImmediateInRangeForLoad" is very misleading. Also it might negatively impact decisions we used to make for other instructions.


+1


http://reviews.llvm.org/D21183





More information about the llvm-commits mailing list