lriggs opened a new pull request, #48038: URL: https://github.com/apache/arrow/pull/48038
### Rationale for this change I believe there is a bug in LLVM 20 where a memory leak can occur on certain architectures. My company's product uses Gandiva and I recently ran into a bug involving Gandiva, the ORC v2/LLJIT compiler and LLVM versions. I was not able to write a unit test to reproduce the bug outside of our code, though I believe there is a high potential for it to reproduce. It is a memory leak, eventually leading to a crash after running several thousand queries. It only occurred on x86 architectures. ### What changes are included in this PR? Adding a VCPKG overlay for LLVM 21 (since its not bundled into an official release yet). The Mac CI builds (in arrow-java for example) use Brew which does not have a 'llvm@21' named package. Installing 'llvm@21' currently resolves to 'llvm' which only provides an llvm library which includes a runtime dependency on Z3. The @ versioned llvms in brew do not do this. This PR will update the mac builds to use vcpkg to install just LLVM in order to fix that and to also ensure parity between the Linux and Mac LLVM versions. Is removing llvm from cpp/Brewfile a problem in this case? Maybe for local mac builds? I'm not sure what to leave here. ### Are these changes tested? TBD ### Are there any user-facing changes? No. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
