[llvm-commits] [Review request] Instcombine: extractvalue from load --> load from gep
Frits van Bommel
fvbommel at gmail.com
Sat Nov 27 20:52:52 CST 2010
On Sun, Nov 28, 2010 at 1:12 AM, Frits van Bommel <fvbommel at gmail.com> wrote:
> The attached patch teaches -instcombine to shrink a non-volatile load
> if the only user is an extractvalue instruction.
> I also corrected a comment that claimed there was nothing left to do
> for extractvalue instructions :).
> Okay to commit?
No, definitely not okay. It inserts the new load at the location of
the extractvalue instead of the location of the old load. Fixed
version attached. I'll provide an extra test case once I've had some
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 2168 bytes
Desc: not available
Url : http://lists.cs.uiuc.edu/pipermail/llvm-commits/attachments/20101128/f58b8ea1/attachment.bin
More information about the llvm-commits