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

David Blaikie dblaikie at gmail.com
Sat Jul 16 23:13:04 PDT 2011


> This *is* highly desirable.


Great - glad I didn't misunderstand that, then.


>  Unfortunately, this breaks all the clients, like clang and dragonegg etc.


Right, so it does. I did fix up clang with this patch (hmm, well I fixed it
up on my machine but forgot that it's in a separate repository - so I've
attached that patch to this email) & happy to checkout dragonegg & fix that
too if it's practical.


>  Can you tweak this so that the main APIs *take* a "const Type" but
> *return* a "Type"?
>

Possibly - with a lot of const casting & I'll have to figure out which parts
of llvm are the actual API used by clients, as opposed to just internal
interfaces.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110716/964d38c4/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: nonconst_type_clang.diff
Type: text/x-patch
Size: 166533 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110716/964d38c4/attachment.bin>


More information about the llvm-commits mailing list