[PATCH] DagCombine sext-ashr-load into ashr-sextload.

[Hal Finkel]

Hi Erik,

-  // fold (sext (and/or/xor (load x), cst)) ->
+  // fold (sext (and/or/xor/srl/sra (load x), cst)) ->
   //      (and/or/xor (sextload x), (sext cst))

You should update the comment specifying the result too.

Also, is this transformation really correct for srl? I don't think it works if the high bit is 1.

 -Hal

----- Original Message -----
> From: "Erik Verbruggen" <erikjv at me.com>
> To: "Commit Messages and Patches for LLVM" <llvm-commits at cs.uiuc.edu>
> Sent: Thursday, August 21, 2014 7:08:30 AM
> Subject: [PATCH] DagCombine sext-ashr-load into ashr-sextload.
> 
> 
> 
> Same as the other mail, but now with the "right" subject line :)
> 
> DagCombine sext-ashr-load into ashr-sextload.
> 
> Same for lshr and zext.
> 
> 
> 
> 
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory