Branch: refs/heads/main
Revision: d36815f
Author: mppf
Link: Improve LLVM configuration on Mac OS X by mppf · Pull Request #18727 · chapel-lang/chapel · GitHub
Log Message:
Merge pull request #18727 from mppf/mac-os-x-link-clang-static
Improve LLVM configuration on Mac OS X
Follow-up to PR #18606
-
Add isysroot argument even for system llvm
- This appears to be necessary on Mac OS X 11.6.1 with Homebrew llvm
version 11.1.0.
- This appears to be necessary on Mac OS X 11.6.1 with Homebrew llvm
-
Link clang statically for Mac OS X with system LLVM
- note that we were already linking statically with the LLVM binaries,
but previously were linking clang dynamically - intended to resolve problems building mason
(https://github.com/Cray/chapel-private/issues/2702)
- note that we were already linking statically with the LLVM binaries,
-
Add -I and -L paths according to sysroot
- The
-Lpath at least seems to be necessary on Mac OS 11.6 and
newer. See Clang linking error MacOS Big Sur | Apple Developer Forums for some
background in this area. Even though the-isysrootpath covers it,
the linking call does not seem to use it.
- The
- [x] homebrew system LLVM 11.1 functions on Big Sur
- [x] homebrew system LLVM 11.1 functions on Catalina
- [x] homebrew system LLVM functions on Mojave
- [x] bundled LLVM functions on Big Sur and Catalina
- [x] full local testing
Reviewed by @ronawho - thanks!
Modified Files:
M third-party/llvm/Makefile.include-system
M util/chplenv/chpl_llvm.py
Compare: https://github.com/chapel-lang/chapel/compare/ed7f03b0c879...d36815f23ca6