[llvm-commits] CVS: llvm/lib/Analysis/AliasSetTracker.cpp

Chris Lattner clattner at apple.com
Sun Sep 17 10:38:54 PDT 2006


> AliasSetTracker.cpp updated: 1.41 -> 1.42
> ---
> Log message:
>
> Fix PR912: http://llvm.org/PR912 . The input to erase() must not be  
> a reference to the data
> being erased.

>      // Finally, remove the entry.
> -    PointerMap.erase(P->first);
> +    Value *Remove = P->first;
> +    PointerMap.erase(Remove);
>    }

Nicholas, this is *excellent* detective work, thanks for tracking  
this down.  Please put a comment here explaining what is going on  
though, because otherwise someone is likely to come along and break  
it again! :)

Thanks again,

-Chris



More information about the llvm-commits mailing list