[llvm-commits] [PATCH] More constant-folding for PPCDoubleDouble

Ulrich Weigand Ulrich.Weigand at de.ibm.com
Tue Oct 30 05:37:11 PDT 2012


Hal Finkel <hfinkel at anl.gov> wrote on 29.10.2012 22:50:56:

> -      APFloat apf(APInt::getNullValue
> (DestTy->getPrimitiveSizeInBits()), true);
> +      APFloat apf(APInt::getNullValue(DestTy->getPrimitiveSizeInBits()),
> +                  !DestTy->isPPC_FP128Ty());
>
> I recommend putting a comment like /* isIEEE */ next to the !
> DestTy->isPPC_FP128Ty() just to remind people (like me) what's going on.

OK, done.

> LGTM.

Checked in as r167013.

Thanks,
Ulrich




More information about the llvm-commits mailing list