From: Dan Gohman Date: Sat, 4 Oct 2008 00:56:36 +0000 (+0000) Subject: Fix fast-isel's handling of atomic instructions. They may X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=241f464d24a6c22721607841069bbeb17b3f71e6;p=oota-llvm.git Fix fast-isel's handling of atomic instructions. They may expand to multiple basic blocks, in which case fast-isel needs to informed of which block to use as it resumes inserting instructions. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@57040 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/llvm/CodeGen/FastISel.h b/include/llvm/CodeGen/FastISel.h index 2b112c878c8..8bf8827481c 100644 --- a/include/llvm/CodeGen/FastISel.h +++ b/include/llvm/CodeGen/FastISel.h @@ -55,12 +55,20 @@ protected: const TargetLowering &TLI; public: + /// startNewBlock - Set the current block, to which generated + /// machine instructions will be appended, and clear the local + /// CSE map. + /// + void startNewBlock(MachineBasicBlock *mbb) { + setCurrentBlock(mbb); + LocalValueMap.clear(); + } + /// setCurrentBlock - Set the current block, to which generated /// machine instructions will be appended. /// void setCurrentBlock(MachineBasicBlock *mbb) { MBB = mbb; - LocalValueMap.clear(); } /// SelectInstruction - Do "fast" instruction selection for the given diff --git a/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp b/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp index f2f14490b16..dce46ab414b 100644 --- a/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp +++ b/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp @@ -756,7 +756,7 @@ void SelectionDAGISel::SelectAllBasicBlocks(Function &Fn, MachineFunction &MF, CodeGenAndEmitDAG(); SDL->clear(); } - FastIS->setCurrentBlock(BB); + FastIS->startNewBlock(BB); // Do FastISel on as many instructions as possible. for (; BI != End; ++BI) { // Just before the terminator instruction, insert instructions to @@ -794,6 +794,9 @@ void SelectionDAGISel::SelectAllBasicBlocks(Function &Fn, MachineFunction &MF, } SelectBasicBlock(LLVMBB, BI, next(BI)); + // If the instruction was codegen'd with multiple blocks, + // inform the FastISel object where to resume inserting. + FastIS->setCurrentBlock(BB); continue; }