diff options
author | Krzysztof Parzyszek <kparzysz@codeaurora.org> | 2017-05-05 22:13:57 +0000 |
---|---|---|
committer | Krzysztof Parzyszek <kparzysz@codeaurora.org> | 2017-05-05 22:13:57 +0000 |
commit | ee93e009c8721eefebc85fcd763fd10343714977 (patch) | |
tree | 0b7e9d984ef90e4043f07bc02a20f2db6ca54335 | |
parent | e260332838c4f70bf03fd7a47662331d1a64920f (diff) | |
download | bcm5719-llvm-ee93e009c8721eefebc85fcd763fd10343714977.tar.gz bcm5719-llvm-ee93e009c8721eefebc85fcd763fd10343714977.zip |
[Hexagon] Disable predicated calls by default
llvm-svn: 302307
-rw-r--r-- | llvm/lib/Target/Hexagon/HexagonInstrInfo.cpp | 10 | ||||
-rw-r--r-- | llvm/lib/Target/Hexagon/HexagonSubtarget.cpp | 8 | ||||
-rw-r--r-- | llvm/lib/Target/Hexagon/HexagonSubtarget.h | 1 |
3 files changed, 18 insertions, 1 deletions
diff --git a/llvm/lib/Target/Hexagon/HexagonInstrInfo.cpp b/llvm/lib/Target/Hexagon/HexagonInstrInfo.cpp index 0971300c962..03794511414 100644 --- a/llvm/lib/Target/Hexagon/HexagonInstrInfo.cpp +++ b/llvm/lib/Target/Hexagon/HexagonInstrInfo.cpp @@ -1467,7 +1467,15 @@ bool HexagonInstrInfo::DefinesPredicate( } bool HexagonInstrInfo::isPredicable(const MachineInstr &MI) const { - return MI.getDesc().isPredicable(); + if (!MI.getDesc().isPredicable()) + return false; + + if (MI.isCall() || isTailCall(MI)) { + const MachineFunction &MF = *MI.getParent()->getParent(); + if (!MF.getSubtarget<HexagonSubtarget>().usePredicatedCalls()) + return false; + } + return true; } bool HexagonInstrInfo::isSchedulingBoundary(const MachineInstr &MI, diff --git a/llvm/lib/Target/Hexagon/HexagonSubtarget.cpp b/llvm/lib/Target/Hexagon/HexagonSubtarget.cpp index d23fc92c5ec..8851a23ae8a 100644 --- a/llvm/lib/Target/Hexagon/HexagonSubtarget.cpp +++ b/llvm/lib/Target/Hexagon/HexagonSubtarget.cpp @@ -73,6 +73,10 @@ static cl::opt<bool> OverrideLongCalls("hexagon-long-calls", cl::Hidden, cl::ZeroOrMore, cl::init(false), cl::desc("If present, forces/disables the use of long calls")); +static cl::opt<bool> EnablePredicatedCalls("hexagon-pred-calls", + cl::Hidden, cl::ZeroOrMore, cl::init(false), + cl::desc("Consider calls to be predicable")); + void HexagonSubtarget::initializeEnvironment() { UseMemOps = false; ModeIEEERndNear = false; @@ -257,6 +261,10 @@ bool HexagonSubtarget::enableMachineScheduler() const { return true; } +bool HexagonSubtarget::usePredicatedCalls() const { + return EnablePredicatedCalls; +} + void HexagonSubtarget::updateLatency(MachineInstr &SrcInst, MachineInstr &DstInst, SDep &Dep) const { if (Dep.isArtificial()) { diff --git a/llvm/lib/Target/Hexagon/HexagonSubtarget.h b/llvm/lib/Target/Hexagon/HexagonSubtarget.h index 647eedc1d12..4379efa79c9 100644 --- a/llvm/lib/Target/Hexagon/HexagonSubtarget.h +++ b/llvm/lib/Target/Hexagon/HexagonSubtarget.h @@ -104,6 +104,7 @@ public: bool useHVXDblOps() const { return UseHVXOps && UseHVXDblOps; } bool useHVXSglOps() const { return UseHVXOps && !UseHVXDblOps; } bool useLongCalls() const { return UseLongCalls; } + bool usePredicatedCalls() const; bool useBSBScheduling() const { return UseBSBScheduling; } bool enableMachineScheduler() const override; |