[PATCH] D113610: Only set LLVM_EXTERNAL_VISIBILITY when building libLLVM dylib

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added a comment.

I tried this config on my Mac and `ninja check-mlir` failed with lot of failures actually:

  Unsupported:  91
  Passed     : 313
  Failed     : 828

I suspect that BUILD_SHARED_LIBS never worked on MacOS here...

(sorry I typed this and didn't hit send before).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113610