[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