[PATCH] use an integer constant for FABS / FNEG (x86)

Andrea Di Biagio andrea.dibiagio at gmail.com
Mon Sep 1 08:13:52 PDT 2014


Hi Sanjay,

the patch LGTM.

On Mon, Sep 1, 2014 at 4:03 PM, Sanjay Patel <spatel at rotateright.com> wrote:
> Ping.
>
> I think it would be ok to commit this small change, but the dev policy page (code review rule #5) says that I need an explicit LGTM because I posted this as a patch for review. Please let me know if I'm misunderstanding the rules.
>
> http://reviews.llvm.org/D5052
>
>
>
> _______________________________________________
> 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