[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