[LLVMdev] PR7407

David A. Greene greened at obbligato.org
Fri Jun 18 11:08:56 PDT 2010


Chris Lattner <clattner at apple.com> writes:

> David, it's possible that Dan just fixed this.  Can you reverify with ToT?

Nope, I just updated and the test still fails for me.  I don't think
Dan's patch addresses the fundamental issue, which is that
ReplaceAllUsesWith deletes nodes, so even if we back out and don't match
to an addressing mode at that point, the node has already been delected
and references to the node up the call chain are now pointing to bogus
memory.

                               -Dave



More information about the llvm-dev mailing list