diff options
| author | David Majnemer <david.majnemer@gmail.com> | 2014-10-04 06:16:45 +0000 |
|---|---|---|
| committer | David Majnemer <david.majnemer@gmail.com> | 2014-10-04 06:16:45 +0000 |
| commit | 7656f418091b1ae51145cf12edabb29fed67ab6d (patch) | |
| tree | c5732240fb77f6c0d82c84d0545eb18defed975c /clang/lib/Sema/SemaDecl.cpp | |
| parent | 808ec85ad0120960734b12c155cda84e9d80f3a7 (diff) | |
| download | bcm5719-llvm-7656f418091b1ae51145cf12edabb29fed67ab6d.tar.gz bcm5719-llvm-7656f418091b1ae51145cf12edabb29fed67ab6d.zip | |
Sema: Simplify checkAttributesAfterMerging
Use getDLLAttr to factor out some common dllimport/dllexport code.
llvm-svn: 219048
Diffstat (limited to 'clang/lib/Sema/SemaDecl.cpp')
| -rw-r--r-- | clang/lib/Sema/SemaDecl.cpp | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp index a3227aa2960..03dd2a7093c 100644 --- a/clang/lib/Sema/SemaDecl.cpp +++ b/clang/lib/Sema/SemaDecl.cpp @@ -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; |
