[PATCH] ARM FastISel fix sext/zext fold

JF Bastien jfb at google.com
Tue Jun 11 15:15:01 PDT 2013


Thanks, committed as r183794.


On Tue, Jun 11, 2013 at 11:52 AM, Chad Rosier <mcrosier at apple.com> wrote:

> 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
> >>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130611/5199375c/attachment.html>


More information about the llvm-commits mailing list