Displaying 5 results from an estimated 5 matches for "oldpr".
Did you mean:
older
2010 Dec 31
3
[LLVMdev] CodeExtractor.cpp potential bug?
There might be a misuse of DominatorTree::splitBasicBlock in
CodeExtractor.cpp, line 145.
Header is splited into two (OldPred->NewBB).
Line 145 updates the dominator tree by calling DT->splitBasicBlock(NewBB).
I think it should be DT->splitBasicBlock(OldPred).
When I tried to extract a code region whose header has 2 successors, my pass
crashed.
It was because header (or OldPred) is the block that was splited,...
2011 Jan 03
0
[LLVMdev] CodeExtractor.cpp potential bug?
On Dec 31, 2010, at 11:20 AM, Vu Le wrote:
> There might be a misuse of DominatorTree::splitBasicBlock in CodeExtractor.cpp, line 145.
> Header is splited into two (OldPred->NewBB).
>
> Line 145 updates the dominator tree by calling DT->splitBasicBlock(NewBB).
> I think it should be DT->splitBasicBlock(OldPred).
>
> When I tried to extract a code region whose header has 2 successors, my pass crashed.
> It was because header (or OldPred)...
2013 Mar 02
2
[LLVMdev] Question about method CodeExtractor::severSplitPHINodes
.../html/classllvm_1_1Value.html#ad452febc1ac0b394876e640ec03ffa38>()+".ce");00216
00217 // We only want to code extract the second block now, and it
becomes the new00218 // header of the region.00219 BasicBlock
<http://llvm.org/docs/doxygen/html/classllvm_1_1BasicBlock.html>
*OldPred = Header;00220 Blocks.remove
<http://llvm.org/docs/doxygen/html/classllvm_1_1SetVector.html#a6357ff5654c7cbe5fed8516dc8328eb4>(OldPred);00221
Blocks.insert
<http://llvm.org/docs/doxygen/html/classllvm_1_1SetVector.html#a72d928b7fc2c5f2d56c6ac0265fd9c6e>(NewBB);00222
Header = New...
2011 Jan 24
0
[LLVMdev] CodeExtractor.cpp potential bug?
...n, Jan 3, 2011 at 1:25 PM, Cameron Zwarich <zwarich at apple.com> wrote:
>
>> On Dec 31, 2010, at 11:20 AM, Vu Le wrote:
>>
>> > There might be a misuse of DominatorTree::splitBasicBlock in
>> CodeExtractor.cpp, line 145.
>> > Header is splited into two (OldPred->NewBB).
>> >
>> > Line 145 updates the dominator tree by calling
>> DT->splitBasicBlock(NewBB).
>> > I think it should be DT->splitBasicBlock(OldPred).
>> >
>> > When I tried to extract a code region whose header has 2 successors, my
&g...
2010 Jan 09
2
[LLVMdev] [PATCH] Fix nondeterministic behaviour in the CodeExtractor
...Block*> BlocksToExtract;
+ SetVector<BasicBlock*> BlocksToExtract;
DominatorTree* DT;
bool AggregateArgs;
unsigned NumExitBlocks;
@@ -135,7 +136,7 @@
// We only want to code extract the second block now, and it becomes the new
// header of the region.
BasicBlock *OldPred = Header;
- BlocksToExtract.erase(OldPred);
+ BlocksToExtract.remove(OldPred);
BlocksToExtract.insert(NewBB);
Header = NewBB;
@@ -180,7 +181,7 @@
}
void CodeExtractor::splitReturnBlocks() {
- for (std::set<BasicBlock*>::iterator I = BlocksToExtract.begin(),
+ for (SetVector&...