small patch to add an assert to PATypeHolder

Project Infrastructure LLVM Dev List Archives
1

Hello,

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.

-e

2

b0fec310cc5450e1b9802a28f02da9fa2a35e02d.diff (384 Bytes)

3

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

-Chris