[llvm-commits] [patch] universally transition to non-const llvm::Type

Chris Lattner clattner at apple.com
Sun Jul 17 21:26:19 PDT 2011


On Jul 16, 2011, at 11:50 PM, David Blaikie wrote:

> happy to checkout dragonegg & fix that too if it's practical.
> 
> Here's a blind attempt (since I don't have gcc source checked out) at fixing dragonegg, too - given the sed replacement worked universally in llvm & clang, I'd say the chances are good that this'll just work. (given my look at dragonegg, it doesn't seem to have any internal types called "Type" (unlike clang, where I only replaced the fully qualified "llvm::Type" due to the existence of "clang::Type"))
> 
> But yeah - if this avenue is still not preferred let me know & I'll look into making the llvm changes without the clang (using const cast on the parameters to llvm functions used by clients) changes & use that (clang's build) to verify that the change doesn't break clients.

Applied in r135371, thanks!  I'll apply your original patch as well after fixing llvm-gcc.

-Chris
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110717/448cb7ad/attachment.html>


More information about the llvm-commits mailing list