[PATCH] D115825: Re-apply "Only define LLVM_EXTERNAL_VISIBILITY when building libLLVM dylib"

[Ben Langmuir via Phabricator via llvm-commits]

benlangmuir added inline comments.


================
Comment at: mlir/include/mlir/Support/TypeID.h:118
 /// available within LLVM.
-struct LLVM_EXTERNAL_VISIBILITY TypeIDExported {
+struct MLIR_CAPI_EXPORTED TypeIDExported {
   template <typename T>
----------------
I'm not sure if matching the CAPI visibility is precisely what you want here, but it seems like a strict improvement over matching the LLVM visibility, which should be mostly orthogonal from what you do here.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D115825/new/

https://reviews.llvm.org/D115825