diff options
| author | Chandler Carruth <chandlerc@gmail.com> | 2014-02-25 09:45:27 +0000 |
|---|---|---|
| committer | Chandler Carruth <chandlerc@gmail.com> | 2014-02-25 09:45:27 +0000 |
| commit | a8c4cc68f5f5f5b1b2dfa86e572623fc45bf6404 (patch) | |
| tree | cea7e2423a72eab3c0960deb706150e8b18f4476 /llvm/lib/Transforms/Scalar | |
| parent | fd88edfddbe75e9ea54cadd211771e27a7671f2c (diff) | |
| download | bcm5719-llvm-a8c4cc68f5f5f5b1b2dfa86e572623fc45bf6404.tar.gz bcm5719-llvm-a8c4cc68f5f5f5b1b2dfa86e572623fc45bf6404.zip | |
[SROA] Fix a *glaring* bug in r202091: you have to actually *write*
the break statement, not just think it to yourself....
No idea how this worked at all, much less survived most bots, my
bootstrap, and some bot bootstraps!
The Polly one didn't survive, and this was filed as PR18959. I don't
have a reduced test case and honestly I'm not seeing the need. What we
probably need here are better asserts / debug-build behavior in
SmallPtrSet so that this madness doesn't make it so far.
llvm-svn: 202129
Diffstat (limited to 'llvm/lib/Transforms/Scalar')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/SROA.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/llvm/lib/Transforms/Scalar/SROA.cpp b/llvm/lib/Transforms/Scalar/SROA.cpp index 107b1b41a84..67903230f36 100644 --- a/llvm/lib/Transforms/Scalar/SROA.cpp +++ b/llvm/lib/Transforms/Scalar/SROA.cpp @@ -3169,6 +3169,7 @@ bool SROA::rewritePartition(AllocaInst &AI, AllocaSlices &S, Promotable = false; PHIUsers.clear(); SelectUsers.clear(); + break; } for (SmallPtrSetImpl<SelectInst *>::iterator I = SelectUsers.begin(), E = SelectUsers.end(); @@ -3177,6 +3178,7 @@ bool SROA::rewritePartition(AllocaInst &AI, AllocaSlices &S, Promotable = false; PHIUsers.clear(); SelectUsers.clear(); + break; } if (Promotable) { |
