This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [cft] rewrite i386 dllimport/dllexport


On Thu, Mar 29, 2007 at 10:07:28PM +1200, Danny Smith wrote:
> In winnt.c: i386_pe_encode_section_info: 
> 
> When generating the decorated name for stdcall/fastcall symbols, your
> patch
> does this:
> +	      XSTR (symbol, 0) = IDENTIFIER_POINTER (newid);
> 
> 'symbol' has not yet been initialised,

I'd managed to find that myself after I posted the patch.

> foo.c:3: warning: 'foo' redeclared without dllimport attribute after
> being referenced with dll linkage
> foo.c:3: internal compiler error: in
> i386_pe_asm_output_aligned_decl_common, at config/i386/winnt.c:487

Fixed by simply removing the assert.  We'll get something random
as the result, and the user will have to fix up their code.  I'm
not sure why we just warn here, instead of hard error.

Committed.


r~


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]