[LLVMdev] small patch to add an assert to PATypeHolder

Chris Lattner sabre at nondot.org
Mon Feb 25 13:31:58 PST 2008


On Mon, 25 Feb 2008, Erick Tryzelaar wrote:
> I ran into an issue with the ocaml bindings where PATypeHolder was getting a
> null pointer, which eventually caused a segfault. This patch copies what
> PATypeHandle does and adds the assert in the PATypeHolder::addRef method.
>
> Also, I'm using git's gateway to subversion to track llvm and send this
> patch. The patch applies fine according to my tests, but if there are any
> problems, I'll find a better way to do this.

Looks great to me, applied, thanks!
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20080225/058766.html

-Chris

-- 
http://nondot.org/sabre/
http://llvm.org/



More information about the llvm-dev mailing list