[PATCH] ARM FastISel fix sext/zext fold

Eric Christopher echristo at gmail.com
Tue Jun 11 11:50:22 PDT 2013


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