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

David Blaikie dblaikie at gmail.com
Sat Jul 16 23:50:35 PDT 2011


>
> 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.

- David
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110716/0875ad72/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: nonconst_type_dragonegg.diff
Type: text/x-patch
Size: 149647 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110716/0875ad72/attachment.bin>


More information about the llvm-commits mailing list