[llvm-commits] [llvm] r159042 - in /llvm/trunk: test/CodeGen/ARM/fusedMAC.ll test/CodeGen/PowerPC/a2-fp-basic.ll test/CodeGen/PowerPC/fma.ll test/CodeGen/PowerPC/ppc440-fp-basic.ll tools/llc/llc.cpp

Chandler Carruth chandlerc at google.com
Sat Jun 23 01:11:35 PDT 2012


On Fri, Jun 22, 2012 at 3:31 PM, Lang Hames <lhames at gmail.com> wrote:

> Author: lhames
> Date: Fri Jun 22 17:31:00 2012
> New Revision: 159042
>
> URL: http://llvm.org/viewvc/llvm-project?rev=159042&view=rev
> Log:
> Rename fp-op fusion option (yet again) for compatibility with GCC option.
>

I'm not interested in any more churn here as I think this bikeshed color
doesn't matter, but going forward, I wouldn't stress so much about the
internal LLVM bits resembling the GCC options. We should name them
sensibly, and leave it to the compatibility driver to munge things
appropriately.


> Modified:
>    llvm/trunk/test/CodeGen/ARM/fusedMAC.ll
>    llvm/trunk/test/CodeGen/PowerPC/a2-fp-basic.ll
>    llvm/trunk/test/CodeGen/PowerPC/fma.ll
>    llvm/trunk/test/CodeGen/PowerPC/ppc440-fp-basic.ll
>    llvm/trunk/tools/llc/llc.cpp
>
> Modified: llvm/trunk/test/CodeGen/ARM/fusedMAC.ll
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/ARM/fusedMAC.ll?rev=159042&r1=159041&r2=159042&view=diff
>
> ==============================================================================
> --- llvm/trunk/test/CodeGen/ARM/fusedMAC.ll (original)
> +++ llvm/trunk/test/CodeGen/ARM/fusedMAC.ll Fri Jun 22 17:31:00 2012
> @@ -1,4 +1,4 @@
> -; RUN: llc < %s -mtriple=armv7-eabi -mattr=+neon,+vfp4 -fuse-fp-ops=fast
> | FileCheck %s
> +; RUN: llc < %s -mtriple=armv7-eabi -mattr=+neon,+vfp4 -fp-contract=fast
> | FileCheck %s
>  ; Check generated fused MAC and MLS.
>
>  define double @fusedMACTest1(double %d1, double %d2, double %d3) {
>
> Modified: llvm/trunk/test/CodeGen/PowerPC/a2-fp-basic.ll
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/PowerPC/a2-fp-basic.ll?rev=159042&r1=159041&r2=159042&view=diff
>
> ==============================================================================
> --- llvm/trunk/test/CodeGen/PowerPC/a2-fp-basic.ll (original)
> +++ llvm/trunk/test/CodeGen/PowerPC/a2-fp-basic.ll Fri Jun 22 17:31:00 2012
> @@ -1,4 +1,4 @@
> -; RUN: llc < %s -march=ppc64 -mcpu=a2 -fuse-fp-ops=fast | FileCheck %s
> +; RUN: llc < %s -march=ppc64 -mcpu=a2 -fp-contract=fast | FileCheck %s
>
>  %0 = type { double, double }
>
>
> Modified: llvm/trunk/test/CodeGen/PowerPC/fma.ll
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/PowerPC/fma.ll?rev=159042&r1=159041&r2=159042&view=diff
>
> ==============================================================================
> --- llvm/trunk/test/CodeGen/PowerPC/fma.ll (original)
> +++ llvm/trunk/test/CodeGen/PowerPC/fma.ll Fri Jun 22 17:31:00 2012
> @@ -1,4 +1,4 @@
> -; RUN: llc < %s -march=ppc32 -fuse-fp-ops=fast | \
> +; RUN: llc < %s -march=ppc32 -fp-contract=fast | \
>  ; RUN:   egrep {fn?madd|fn?msub} | count 8
>
>  define double @test_FMADD1(double %A, double %B, double %C) {
>
> Modified: llvm/trunk/test/CodeGen/PowerPC/ppc440-fp-basic.ll
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/PowerPC/ppc440-fp-basic.ll?rev=159042&r1=159041&r2=159042&view=diff
>
> ==============================================================================
> --- llvm/trunk/test/CodeGen/PowerPC/ppc440-fp-basic.ll (original)
> +++ llvm/trunk/test/CodeGen/PowerPC/ppc440-fp-basic.ll Fri Jun 22 17:31:00
> 2012
> @@ -1,4 +1,4 @@
> -; RUN: llc < %s -march=ppc32 -mcpu=440 -fuse-fp-ops=fast | FileCheck %s
> +; RUN: llc < %s -march=ppc32 -mcpu=440 -fp-contract=fast | FileCheck %s
>
>  %0 = type { double, double }
>
>
> Modified: llvm/trunk/tools/llc/llc.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llc/llc.cpp?rev=159042&r1=159041&r2=159042&view=diff
>
> ==============================================================================
> --- llvm/trunk/tools/llc/llc.cpp (original)
> +++ llvm/trunk/tools/llc/llc.cpp Fri Jun 22 17:31:00 2012
> @@ -195,15 +195,15 @@
>     clEnumValEnd));
>
>  static cl::opt<llvm::FPOpFusion::FPOpFusionMode>
> -FuseFPOps("fuse-fp-ops",
> +FuseFPOps("fp-contract",
>   cl::desc("Enable aggresive formation of fused FP ops"),
>   cl::init(FPOpFusion::Standard),
>   cl::values(
>     clEnumValN(FPOpFusion::Fast, "fast",
>                "Fuse FP ops whenever profitable"),
> -    clEnumValN(FPOpFusion::Standard, "standard",
> +    clEnumValN(FPOpFusion::Standard, "on",
>                "Only fuse 'blessed' FP ops."),
> -    clEnumValN(FPOpFusion::Strict, "strict",
> +    clEnumValN(FPOpFusion::Strict, "off",
>                "Only fuse FP ops when the result won't be effected."),
>     clEnumValEnd));
>
>
>
> _______________________________________________
> 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/20120623/4177c6c1/attachment.html>


More information about the llvm-commits mailing list