[PATCH] [Sema] Re-land "Make FunctionType's TSI use unadjusted argument types"

Eli Friedman eli.friedman at gmail.com
Tue Jun 11 14:24:11 PDT 2013


  At a higher level, the big, potentially breaking change here is that you're making FunctionProtoType::getArgType() potentially return an undecayed type.  There are around 100 users of this API in-tree, and it looks like a substantial fraction of them are going to need a similar change.  I'm not comfortable with an API named getArgType() returning an un-decayed type; even if you catch all the existing cases which need additional checks, people writing new code aren't going to be thinking about this, and the un-decayed type is almost always the wrong answer from a semantic point of view.  Any APIs on FunctionProtoType returning un-decayed types should have "un-decayed" in the name, IMO.

http://llvm-reviews.chandlerc.com/D957



More information about the cfe-commits mailing list