[cfe-dev] Implement type merging in ASTContext

Chris Lattner clattner at apple.com
Thu Aug 21 15:44:00 PDT 2008


On Aug 19, 2008, at 7:27 PM, Eli Friedman wrote:

> Patch per subject, adds a method to query for the merged type for two
> types.  This is necessary in a few places per the C standard, like
> redeclarations and conditionals, although the effect is usually pretty
> subtle.

Ok

> I think this patch is pretty close to correct; the one thing I'm
> really wondering about is ObjC types.  I don't know what the rules for
> compatibility for ObjC types are supposed to be, so the current patch
> is mostly just guessing.  The version attached doesn't cause any
> regressions in the testsuite, so I think this gets the basics correct,
> but I might be missing some cases.  If anyone actually knows the
> complete rules, I'd appreciate a patch review or a pointer to the
> rules.

It looks pretty reasonable to me, but I didn't scrutinize the logic.   
If you think it is a step forward in the current codebase (towards  
correctness) plz feel free to commit it, even if not perfect.

-Chris



More information about the cfe-dev mailing list