diff options
| author | Florian Hahn <florian.hahn@arm.com> | 2017-06-19 13:45:41 +0000 |
|---|---|---|
| committer | Florian Hahn <florian.hahn@arm.com> | 2017-06-19 13:45:41 +0000 |
| commit | fd44ca6c76c1e9ae0a934fc217cdb62504d9dd97 (patch) | |
| tree | ed5fd38483c3fc6cd6ad91b4edc0986ba8969984 /llvm/lib/Target | |
| parent | 48bed539187e06e40a31fa7ef3c574ec445ef5bd (diff) | |
| download | bcm5719-llvm-fd44ca6c76c1e9ae0a934fc217cdb62504d9dd97.tar.gz bcm5719-llvm-fd44ca6c76c1e9ae0a934fc217cdb62504d9dd97.zip | |
[AArch64] Fix order of checks in shouldScheduleAdjacent.
We need to check the opcode of FirstMI before accessing the operands. This
caused a buildbot failure during bootstrapping on AArch64.
llvm-svn: 305694
Diffstat (limited to 'llvm/lib/Target')
| -rw-r--r-- | llvm/lib/Target/AArch64/AArch64MacroFusion.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Target/AArch64/AArch64MacroFusion.cpp b/llvm/lib/Target/AArch64/AArch64MacroFusion.cpp index 7e97b6ccb48..ccc9d2ad1b4 100644 --- a/llvm/lib/Target/AArch64/AArch64MacroFusion.cpp +++ b/llvm/lib/Target/AArch64/AArch64MacroFusion.cpp @@ -144,8 +144,8 @@ static bool shouldScheduleAdjacent(const TargetInstrInfo &TII, return FirstOpcode == AArch64::INSTRUCTION_LIST_END || (FirstOpcode == AArch64::MOVZXi && SecondMI.getOperand(3).getImm() == 16) || - (FirstMI->getOperand(3).getImm() == 32 && - FirstOpcode == AArch64::MOVKXi && + (FirstOpcode == AArch64::MOVKXi && + FirstMI->getOperand(3).getImm() == 32 && SecondMI.getOperand(3).getImm() == 48); } |
