diff options
author | Ahmed Bougacha <ahmed.bougacha@gmail.com> | 2017-02-24 00:34:41 +0000 |
---|---|---|
committer | Ahmed Bougacha <ahmed.bougacha@gmail.com> | 2017-02-24 00:34:41 +0000 |
commit | 8f9e99bcb6558846b85202029c137d6d72350530 (patch) | |
tree | cd997a4b4220fb6d3f49b7636f2416170d8b35a8 /llvm/lib/CodeGen | |
parent | ecc6de35fba933b1081a7847b3687f2e99b3e34c (diff) | |
download | bcm5719-llvm-8f9e99bcb6558846b85202029c137d6d72350530.tar.gz bcm5719-llvm-8f9e99bcb6558846b85202029c137d6d72350530.zip |
[GlobalISel] Remove now-unnecessary variable. NFC.
Since r296047, we're able to return early on failures.
Don't track whether we succeeded.
llvm-svn: 296057
Diffstat (limited to 'llvm/lib/CodeGen')
-rw-r--r-- | llvm/lib/CodeGen/GlobalISel/IRTranslator.cpp | 29 |
1 files changed, 14 insertions, 15 deletions
diff --git a/llvm/lib/CodeGen/GlobalISel/IRTranslator.cpp b/llvm/lib/CodeGen/GlobalISel/IRTranslator.cpp index 6b93a121ce3..2df3810afae 100644 --- a/llvm/lib/CodeGen/GlobalISel/IRTranslator.cpp +++ b/llvm/lib/CodeGen/GlobalISel/IRTranslator.cpp @@ -1048,8 +1048,7 @@ bool IRTranslator::runOnMachineFunction(MachineFunction &CurMF) { SmallVector<unsigned, 8> VRegArgs; for (const Argument &Arg: F.args()) VRegArgs.push_back(getOrCreateVReg(Arg)); - bool Succeeded = CLI->lowerFormalArguments(EntryBuilder, F, VRegArgs); - if (!Succeeded) { + if (!CLI->lowerFormalArguments(EntryBuilder, F, VRegArgs)) { OptimizationRemarkMissed R("gisel-irtranslator", "GISelFailure", DebugLoc(), &MF->getFunction()->getEntryBlock()); R << "unable to lower arguments: " << ore::NV("Prototype", F.getType()); @@ -1065,19 +1064,19 @@ bool IRTranslator::runOnMachineFunction(MachineFunction &CurMF) { CurBuilder.setMBB(MBB); for (const Instruction &Inst: BB) { - Succeeded &= translate(Inst); - if (!Succeeded) { - std::string InstStrStorage; - raw_string_ostream InstStr(InstStrStorage); - InstStr << Inst; - - OptimizationRemarkMissed R("gisel-irtranslator", "IRTranslatorFailure: ", - &Inst); - R << "unable to translate instruction: " << ore::NV("Opcode", &Inst) - << ": '" << InstStr.str() << "'"; - reportTranslationError(*MF, *TPC, *ORE, R); - return false; - } + if (translate(Inst)) + continue; + + std::string InstStrStorage; + raw_string_ostream InstStr(InstStrStorage); + InstStr << Inst; + + OptimizationRemarkMissed R("gisel-irtranslator", "IRTranslatorFailure: ", + &Inst); + R << "unable to translate instruction: " << ore::NV("Opcode", &Inst) + << ": '" << InstStr.str() << "'"; + reportTranslationError(*MF, *TPC, *ORE, R); + return false; } } |