<br>Yeah, for us who have already made the changes, this is  kinda bitter sweet.<br><br>Maurice<br><br><div class="gmail_quote">On Fri, Jul 24, 2009 at 2:08 PM, Richard Pennington <span dir="ltr"><<a href="mailto:rich@pennware.com">rich@pennware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">I hate to say it (because I already made the changes), but keeping<br>
compatibility with 2.5 is a good thing.<br>
<br>
-Rich<br>
<div><div></div><div class="h5"><br>
Owen Anderson wrote:<br>
> So, as you all probably noticed, the APIs for creating constants have<br>
> been moved (or in a specific cases, are about to be moved) from static<br>
> methods on Constant to instance methods on LLVMContext.<br>
><br>
> However, as was recently pointed out on llvm-commits and on IRC, this is<br>
> no longer strictly necessary.  Because types have contexts, all of the<br>
> constant subclasses could be friended to LLVMContext, and then put the<br>
> work of uniquing by accessing the context off of the type.<br>
><br>
> There are obviously some pros and to both approaches.  Going back to the<br>
> old-style increases compatibility with 2.5, and breaks up the massive<br>
> LLVMContext class.  On the downside, it would cause massive thrashing on<br>
> TOT, because I will have to revert about two weeks worth of commits in<br>
> widely dispersed parts of the tree, and because a number of clients have<br>
> already switched to the new TOT API.<br>
><br>
> Any comments and/or opinions on this decision are welcome.<br>
><br>
> --Owen<br>
><br>
><br>
</div></div>> ------------------------------------------------------------------------<br>
<div><div></div><div class="h5">><br>
> _______________________________________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</div></div></blockquote></div><br>