[llvm] r213511 - [mips] Do not emit '.module [no]oddspreg' unless we really need to.
Daniel Sanders
daniel.sanders at imgtec.com
Mon Jul 21 03:45:47 PDT 2014
Author: dsanders
Date: Mon Jul 21 05:45:47 2014
New Revision: 213511
URL: http://llvm.org/viewvc/llvm-project?rev=213511&view=rev
Log:
[mips] Do not emit '.module [no]oddspreg' unless we really need to.
We now emit this directive when we need to contradict the default value (e.g.
-mno-odd-spreg is given) or an option changed the default value (e.g. -mfpxx
is given).
This restores support for the currently available head of binutils. However,
at this point binutils 2.24 is still not sufficient since it does not support
'.module fp=...'.
Modified:
llvm/trunk/lib/Target/Mips/MipsAsmPrinter.cpp
llvm/trunk/test/CodeGen/Mips/no-odd-spreg.ll
Modified: llvm/trunk/lib/Target/Mips/MipsAsmPrinter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsAsmPrinter.cpp?rev=213511&r1=213510&r2=213511&view=diff
==============================================================================
--- llvm/trunk/lib/Target/Mips/MipsAsmPrinter.cpp (original)
+++ llvm/trunk/lib/Target/Mips/MipsAsmPrinter.cpp Mon Jul 21 05:45:47 2014
@@ -710,9 +710,14 @@ void MipsAsmPrinter::EmitStartOfAsmFile(
getTargetStreamer().updateABIInfo(*Subtarget);
getTargetStreamer().emitDirectiveModuleFP();
- if (Subtarget->isABI_O32())
- getTargetStreamer().emitDirectiveModuleOddSPReg(Subtarget->useOddSPReg(),
- Subtarget->isABI_O32());
+ // If we are targeting O32 then we must emit a '.module [no]oddspreg' ...
+ if (Subtarget->isABI_O32()) {
+ // ... but don't emit it unless we are contradicting the default or an
+ // option has changed the default (i.e. FPXX).
+ if (!Subtarget->useOddSPReg() || Subtarget->isABI_FPXX())
+ getTargetStreamer().emitDirectiveModuleOddSPReg(Subtarget->useOddSPReg(),
+ Subtarget->isABI_O32());
+ }
}
void MipsAsmPrinter::EmitJal(MCSymbol *Symbol) {
Modified: llvm/trunk/test/CodeGen/Mips/no-odd-spreg.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/Mips/no-odd-spreg.ll?rev=213511&r1=213510&r2=213511&view=diff
==============================================================================
--- llvm/trunk/test/CodeGen/Mips/no-odd-spreg.ll (original)
+++ llvm/trunk/test/CodeGen/Mips/no-odd-spreg.ll Mon Jul 21 05:45:47 2014
@@ -1,10 +1,14 @@
-; RUN: llc -march=mipsel -mcpu=mips32 < %s | FileCheck %s -check-prefix=ALL -check-prefix=ODDSPREG
+; RUN: llc -march=mipsel -mcpu=mips32 < %s | FileCheck %s -check-prefix=ALL -check-prefix=ODDSPREG -check-prefix=ODDSPREG-NO-EMIT
; RUN: llc -march=mipsel -mcpu=mips32 -mattr=+nooddspreg < %s | FileCheck %s -check-prefix=ALL -check-prefix=NOODDSPREG
-; RUN: llc -march=mipsel -mcpu=mips32r6 -mattr=fp64 < %s | FileCheck %s -check-prefix=ALL -check-prefix=ODDSPREG
+; RUN: llc -march=mipsel -mcpu=mips32r6 -mattr=fp64 < %s | FileCheck %s -check-prefix=ALL -check-prefix=ODDSPREG -check-prefix=ODDSPREG-NO-EMIT
; RUN: llc -march=mipsel -mcpu=mips32r6 -mattr=fp64,+nooddspreg < %s | FileCheck %s -check-prefix=ALL -check-prefix=NOODDSPREG
+; RUN: llc -march=mipsel -mcpu=mips32r6 -mattr=fpxx,-nooddspreg < %s | FileCheck %s -check-prefix=ALL -check-prefix=ODDSPREG -check-prefix=ODDSPREG-EMIT
-; ODDSPREG: .module oddspreg
-; NOODDSPREG: .module nooddspreg
+; We don't emit a directive unless we need to. This is to support versions of
+; GAS which do not support the directive.
+; ODDSPREG-EMIT: .module oddspreg
+; ODDSPREG-NO-EMIT-NOT: .module oddspreg
+; NOODDSPREG: .module nooddspreg
define float @two_floats(float %a) {
entry:
More information about the llvm-commits
mailing list