[PATCH] ARM FastISel fix sext/zext fold

Chad Rosier mcrosier at apple.com
Tue Jun 11 11:52:18 PDT 2013


Eric is correct.  Sorry for the belated response.

 Chad

On Jun 11, 2013, at 11:50 AM, Eric Christopher <echristo at gmail.com> wrote:

> I'd assume that Chad's LGTM is still valid after the slight style
> changes. LGTM too.
> 
> -eric
> 
> On Mon, Jun 10, 2013 at 5:35 PM, JF Bastien <jfb at google.com> wrote:
>> Here's an updated patch with the array in an anonymous namespace, with a
>> name on the struct type, and a better description.
>> 
>> I also fixed i < e to i != e as pointed out off-list by Stephen.
>> 
>> Chad, Still LGTY?
>> 
>> _______________________________________________
>> 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