Patch to fix an invalid AST of a locally redeclared built-in which causes an IRGen crash

[jahanian]

On May 21, 2014, at 5:19 PM, Richard Smith <richard at metafoo.co.uk> wrote:

> 
> How about this: drop the setInvalidDecl() call. In its place, when we get a type mismatch between two function declarations, if Old is implicit then check whether it used to be a builtin. If so, check the type against the most recent declaration instead.

Without adding an extra flag to FunctionDecl, I am not sure how I can check that an implicit Old used to be a “builtin”?
I don’t thing that checking for Builtin::NotBuiltin means this as it is a value of 0. Are you proposing
addition of a new flag? Or a special BID?

- fariborz


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140522/1d2077af/attachment.html>