llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: 徐持恒 Xu Chiheng (xu-chiheng) <details> <summary>Changes</summary> --- Full diff: https://github.com/llvm/llvm-project/pull/74982.diff 1 Files Affected: - (modified) clang/include/clang/Interpreter/Value.h (+2-2) ``````````diff diff --git a/clang/include/clang/Interpreter/Value.h b/clang/include/clang/Interpreter/Value.h index c380cd91550def..46e827f3ec283d 100644 --- a/clang/include/clang/Interpreter/Value.h +++ b/clang/include/clang/Interpreter/Value.h @@ -52,7 +52,8 @@ class ASTContext; class Interpreter; class QualType; -#if defined(_WIN32) +#if defined(LLVM_BUILD_LLVM_DYLIB) || defined(LLVM_BUILD_SHARED_LIBS) +#if defined(_WIN32) && !(defined(__CYGWIN__) || defined(__MINGW32__)) // REPL_EXTERNAL_VISIBILITY are symbols that we need to be able to locate // at runtime. On Windows, this requires them to be exported from any of the // modules loaded at runtime. Marking them as dllexport achieves this; both @@ -63,7 +64,6 @@ class QualType; // statically linked into an EXE, it makes sure that they're exported. #define REPL_EXTERNAL_VISIBILITY __declspec(dllexport) #elif __has_attribute(visibility) -#if defined(LLVM_BUILD_LLVM_DYLIB) || defined(LLVM_BUILD_SHARED_LIBS) #define REPL_EXTERNAL_VISIBILITY __attribute__((visibility("default"))) #else #define REPL_EXTERNAL_VISIBILITY `````````` </details> https://github.com/llvm/llvm-project/pull/74982 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
