On Tue, May 31, 2011 at 1:02 PM, John McCall <[email protected]> wrote: > On May 31, 2011, at 4:44 AM, Francois Pichet wrote: >> Author: fpichet >> Date: Tue May 31 06:44:00 2011 >> New Revision: 132332 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=132332&view=rev >> Log: >> For compatibility with MSVC, a friend declaration also act as a forward >> declaration if the tag name is not already declared. The tag name is >> declared in the next outermost non record scope. > > This isn't necessary; there's already logic in ActOnTag to inject a > declaration of the class in the correct scope. You just need to change it > from "invisible unless already declared" to "always visible" in MSVC mode. >
Ok thank for pointing that out. My new patch r132387 is now 2 lines. _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
