On Jun 28, 2012, at 1:39 PM, Timur Iskhodzhanov wrote:
> Part one is attached - it propagates the "isInsance" boolean to the
> two places where it should be read afterwards [in part 2].
I didn't really like this approach, so I committed a different one as r159893.
Let me know if this helps.
> The only logic change is around lib/CodeGen/CGCall.cpp:150.
> Without this change, the definition of `void __cdecl cdecl_method() {}`
> ends up using "thiscall" (and the call remains "cdecl" - ouch!).
> Probably it was a bug in the first place.
Let me know if this is still a problem arising in your second patch.
> Repeating the question I originally had:
> Can you please tell me if the extra isInstance bool parameter to
> arrangeFunctionType is fine or I should rather add this boolean
> into FunctionType::ExtInfo and set it in AST?
The FunctionType should just preserve whether or not an explicit CC
was set and should not try to infer instance-ness.
John.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
