[LLVMdev] [type-system-rewrite][PATCH] de-constify Type* in FunctionType::get()

Chris Lattner clattner at apple.com
Tue Jul 5 12:58:07 PDT 2011


Sure, go for it.  

FWIW, I was waiting to do this (eventually "const Type" will just be "Type" everywhere).

I have a long list of cleanups that I want to land after the branch lands... but I was waiting for after the branch lands to make merging it less annoying and make things more incremental.  Unfortunately I'm struggling to find time to finish things up and land the branch (this week for sure! :).  Since you've already done this work, go ahead and land it if you'd like.

-Chris

On Jul 5, 2011, at 2:48 AM, Jay Foad wrote:

> This patch removes the deprecated form of FunctionType::get() which
> takes "const Type*"s for the argument types, and replaces just enough
> "const Type*" with "Type*" elsewhere to get everything building
> cleanly again. (My motivation for this was that I was getting
> ambiguous overload errors from some calls to FunctionType::get() in
> the Clang source.)
> 
> Chris, are you interested in having this committed on the branch?
> 
> Thanks,
> Jay.
> <deconstify-type.diff>




More information about the llvm-dev mailing list