On Wed, Feb 1, 2012 at 9:33 PM, Eli Friedman <[email protected]> wrote: > On Wed, Feb 1, 2012 at 9:13 PM, Anton Yartsev <[email protected]> wrote: >> Author: ayartsev >> Date: Wed Feb 1 23:13:59 2012 >> New Revision: 149587 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=149587&view=rev >> Log: >> Fix for PR10657 (http://llvm.org/bugs/show_bug.cgi?id=10657) >> extern inline case considered >> >> Modified: >> cfe/trunk/lib/AST/ASTContext.cpp >> cfe/trunk/test/CodeGen/inline.c >> cfe/trunk/test/CodeGen/inline2.c > > The changes you are making to the tests are clearly wrong. Please > revert, and we can discuss the correct way to fix your testcase.
I just took a quick look; a correct patch would probably involve changing FunctionDecl::doesDeclarationForceExternallyVisibleDefinition . -Eli _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
