This revision was automatically updated to reflect the committed changes. Closed by commit rG0a54c325beff: Add note about inlining dllimport functions to the attribute docs (authored by hans).
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115252/new/ https://reviews.llvm.org/D115252 Files: clang/include/clang/Basic/AttrDocs.td Index: clang/include/clang/Basic/AttrDocs.td =================================================================== --- clang/include/clang/Basic/AttrDocs.td +++ clang/include/clang/Basic/AttrDocs.td @@ -187,6 +187,10 @@ are imported from external modules. See the dllimport_ documentation on MSDN for more information. +Note that a dllimport function may still be inlined, if its definition is +available and it doesn't reference any non-dllimport functions or global +variables. + .. _dllimport: https://msdn.microsoft.com/en-us/library/3y1sfaz2.aspx }]; }
Index: clang/include/clang/Basic/AttrDocs.td =================================================================== --- clang/include/clang/Basic/AttrDocs.td +++ clang/include/clang/Basic/AttrDocs.td @@ -187,6 +187,10 @@ are imported from external modules. See the dllimport_ documentation on MSDN for more information. +Note that a dllimport function may still be inlined, if its definition is +available and it doesn't reference any non-dllimport functions or global +variables. + .. _dllimport: https://msdn.microsoft.com/en-us/library/3y1sfaz2.aspx }]; }
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
