[PATCH] R600/SI: Fix various operand legalization bugs

Tom Stellard tom at stellard.net
Tue Sep 23 18:52:32 PDT 2014


On Tue, Sep 23, 2014 at 10:55:26AM -0700, Matt Arsenault wrote:
> Hi,
> 
> This set of patches works towards removing SIISelLowering::legalizeOperands in favor of a post-isel hook which uses SIInstrInfo::legalizeOperands, which requires most of the same logic anyway. As in intermediate step, keep most of the original code in place for immediate folding where it is. This also fixes the overly strict enforcement of the 1 SGPR use constraint, which has been preventing f64 fdiv from working correctly. 
> 

Hi,

Patches 1,2,5,and 6 look good to me.  I will try to review the others
when I have more time.

-Tom











> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list