Sema: Simplify checkAttributesAfterMerging
Use getDLLAttr to factor out some common dllimport/dllexport code. llvm-svn: 219048
This commit is contained in:
parent
808ec85ad0
commit
7656f41809
|
@ -5103,14 +5103,7 @@ static void checkAttributesAfterMerging(Sema &S, NamedDecl &ND) {
|
|||
}
|
||||
|
||||
// dll attributes require external linkage.
|
||||
if (const DLLImportAttr *Attr = ND.getAttr<DLLImportAttr>()) {
|
||||
if (!ND.isExternallyVisible()) {
|
||||
S.Diag(ND.getLocation(), diag::err_attribute_dll_not_extern)
|
||||
<< &ND << Attr;
|
||||
ND.setInvalidDecl();
|
||||
}
|
||||
}
|
||||
if (const DLLExportAttr *Attr = ND.getAttr<DLLExportAttr>()) {
|
||||
if (const InheritableAttr *Attr = getDLLAttr(&ND)) {
|
||||
if (!ND.isExternallyVisible()) {
|
||||
S.Diag(ND.getLocation(), diag::err_attribute_dll_not_extern)
|
||||
<< &ND << Attr;
|
||||
|
|
Loading…
Reference in New Issue