[LLVMdev] RFC: Constant Creation API

Owen Anderson resistor at mac.com
Thu Jul 23 19:35:19 PDT 2009


So, as you all probably noticed, the APIs for creating constants have  
been moved (or in a specific cases, are about to be moved) from static  
methods on Constant to instance methods on LLVMContext.

However, as was recently pointed out on llvm-commits and on IRC, this  
is no longer strictly necessary.  Because types have contexts, all of  
the constant subclasses could be friended to LLVMContext, and then put  
the work of uniquing by accessing the context off of the type.

There are obviously some pros and to both approaches.  Going back to  
the old-style increases compatibility with 2.5, and breaks up the  
massive LLVMContext class.  On the downside, it would cause massive  
thrashing on TOT, because I will have to revert about two weeks worth  
of commits in widely dispersed parts of the tree, and because a number  
of clients have already switched to the new TOT API.

Any comments and/or opinions on this decision are welcome.

--Owen
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 2620 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20090723/cbd2a78c/attachment.bin>


More information about the llvm-dev mailing list