[llvm-commits] [llvm] r52459 - in /llvm/trunk: lib/Transforms/IPO/DeadArgumentElimination.cpp test/Transforms/DeadArgElim/multdeadretval.ll

[Matthijs Kooijman]

Hi Dan,

> I investigated this a little and found the subtle bug in this
> bit of code that's triggering the infinite looping behavior.
> The erase call is invalidating the Range.second iterator in
> a MarkLive call earlier in the recursion. I guess your
> original code which used lower_bound and just tested the
> key at each iteration was a better approach after all, and I
> shouldn't have suggested you change it :-}.
Ah, right. I've indeed reverted to the old code, which helps in the SPAS
testcase. I'm currently running a full test-suite, I'll have results in the
morning.

Gr.

Matthijs
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20080619/91f0ad9e/attachment.sig>