llvm: 10.0.1_2

[terencehonles]

Copies llvm.rb at b32d4a6 as [email protected]

• Have you followed the guidelines for contributing?
• Have you checked that there aren't other open pull requests for the same formula update/change?
• Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
• Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
• Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

---

[terencehonles]

@fxcoudert not sure if you mind reviewing this. You were the one to change llvm.rb most recently and I noticed there was no llvm@10. I don't use a Mac, but I believe this is what is probably needed and missed when converting the formula to llvm 11.

[fxcoudert]

Here are our criteria for versioned formulas in Homebrew: https://docs.brew.sh/Versions
Please comment on how whether they are satisfied for this formula.

In particular:

Upstream should have a release branch for each formula version, and release security updates for each version when necessary

[terencehonles]

Here are our criteria for versioned formulas in Homebrew: https://docs.brew.sh/Versions
Please comment on how whether they are satisfied for this formula.

In particular:

Upstream should have a release branch for each formula version, and release security updates for each version when necessary

This is for LLVM, there is already llvm@6, llvm@7, llvm@8, and llvm@9 so I expect the answer to your question is yes, but you tell me. As mentioned I'm not a Mac/homebrew user. I am just trying to unblock apache/arrow#8386 until apache/arrow#8492 which you opened is actually supported (It looks like there are LLVM changes needed to arrow's build system)

[fxcoudert]

Thanks for proposing the pull request.

• We've fixed Arrow in homebrew-core by switching it to LLVM 9 (which we provide, as you noted).
• Older LLVM versions were needed because other software relied on them. Let's not introduce yet another LLVM version.

[terencehonles]

Thanks for proposing the pull request.

• We've fixed Arrow in homebrew-core by switching it to LLVM 9 (which we provide, as you noted).
• Older LLVM versions were needed because other software relied on them. Let's not introduce yet another LLVM version.

Ok, I'll try updating arrow's Brewfile to use llvm@9 the same way you are suggesting you did. Thanks for the update.