Hi, nice patch.<br><br><br>NewFD->setVirtualAsWritten(true);<br><br>Do we need to set this on the method? According to the docs, this should only be set if virtual was explicitly written in the code.<br><br><br><br><div>

-  if (!isDefinition || (NewTag != TTK_Class && NewTag != TTK_Struct))<br>-    if (OldTag == NewTag)<br>-      return true;<br>+  if (!isDefinition || !isClassCompatTagKind(NewTag) || OldTag == NewTag)<br>+    return true;<br>

<br>The behaviour of the code is changed here, though I'm not sure if that has any impact on correctness.<br><br><br><br>+    default: assert("Invalid tag kind for field padding diagnostic!");<br><br>You used this assert message on a couple diagnostic index helper functions which is incorrect for some. Seems to be a simple copy paste mismatch.</div>

<div><br></div><div><br>+    case TTK_Interface:<br>       Out << 'U';<br><br>Did you confirm that cl mangles interfaces as structs?<br><br><br>+      case TTK_Interface:<br>+      case TTK_Struct: return CXCursor_StructDecl;<br>

<br>I think we should add a new cursor in the C API, and not re-use CXCursor_StructDecl for interfaces.</div><div><br></div><div><br><br>On Sat, Aug 18, 2012 at 5:40 AM, David Robins <<a href="mailto:llvm@davidrobins.net">llvm@davidrobins.net</a>> wrote:<br>

><br>> This is a rework of the previous patch I posted, this time with<br>> __interface as a class-key rather than struct with an attribute.<br>><br>> All tests are currently passing, and I modified the Microsoft extension<br>

> test to account for some new capabilities.<br>><br>> I do not have commit access, so if this patch is acceptable I will need<br>> someone else to commit it. If changes are required first, I am as before<br>
> happy to make them, and to later support this addition.<br>
><br>><br>> _______________________________________________<br>> cfe-dev mailing list<br>> <a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>

><br><br><br><br>--<br>Joćo Matos</div>