[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