[PATCH] Sema: Check dll attributes on redeclarations

Nico Rieck nico.rieck at gmail.com
Sun Mar 30 11:05:35 PDT 2014


On 30.03.2014 19:35, Reid Kleckner wrote:
> Should we be more specific here?  Something like:
> redeclaration %0 adds attribute %1 which was missing the previous
> declaration

Sounds like a good idea. I would phrase it: "redeclaration of %q0 may
not add %q1 attribute" (or rather "cannot"?) which would look like:

t.cpp:222:28: error: redeclaration of 'X::f' may not add 'dllexport'
attribute
__declspec(dllexport) void X::f() {}
                              ^
t.cpp:219:6: note: previous declaration is here
  void f();




More information about the cfe-commits mailing list