[llvm-commits] [patch] universally transition to non-const llvm::Type
Chris Lattner
clattner at apple.com
Sun Jul 17 21:25:03 PDT 2011
On Jul 16, 2011, at 11:13 PM, David Blaikie wrote:
>
> 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.
> <nonconst_type_clang.diff>
Thanks, applied in r135370!
-Chris
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110717/743f2fb7/attachment.html>
More information about the llvm-commits
mailing list