llvm/clang: optionally enable the sanitizers

[aktau]

The clang sanitizers are amazingly useful for detecting all sorts of problems with the code. And it would be cool to have this (optionally) in the standard llvm build.

It's in the compiler-rt git repository and is already optionally installed by the homebrew/versions/llvm34 brew: http://compiler-rt.llvm.org/ so it's already been tested.

• homebrew/versions/llvm34.rb

Look at the --with-asan switch. This actually enables all sanitizers, not just asan.

[adamv]

This should be opened at homebrew-versions: https://github.com/Homebrew/homebrew-versions

[aktau]

@adamv as I had mentioned in my original post, the formula's in llvm-versions already have the option requested, so no need to open an issue there. I opened it here so that the standard llvm build could also have it. Is there a good reason not to include this option?

[adamv]

@Homebrew/owners what's the reason we still package llvm 3.4 here instead of 3.5?

[jacknagel]

3.4 is the current release

[adamv]

The mind boggles.

Can we at least pull the 3.4 formula from versions over, as it is more full-featured, or are we deliberately packaging a stripped down formula? There are some other LLVM issues open asking for features that look to already be provided in the versions version.

[jacknagel]

It is my understanding that the (keg-only) llvm in core exists mostly for things that depend on it, like language runtimes, and the (non-keg-only) homebrew-versions are intended more for direct use.

I'd suggest not changing anything, I don't really want to start shuttling these things back and forth between taps.

[adamv]

Closing; the recommendation is to use the more full-featured LLVMs from versions if you need additional features.