[llvm-commits] [llvm] r168181 - in /llvm/trunk: lib/Transforms/InstCombine/InstCombineCompares.cpp test/Transforms/InstCombine/icmp.ll

[Duncan Sands]

Hi Takumi,

On 22/11/12 10:58, NAKAMURA Takumi wrote:
> Duncan, thanks.
>
>> I don't think having the build be warning free is part of the release
>> criteria.
>
> http://llvm.org/docs/HowToReleaseLLVM.html#release-build
>
>> The builds of llvm, clang, and dragonegg *must* be free of errors and warnings in Debug, Release+Asserts, and Release builds. If all builds are clean, then the release passes Build Qualification.

this criterion hasn't actually been applied in practice for the last few
releases...

>
> Now with our mature clang, I think we should keep the tree
> warnings-clean to prove clang itself. ;)

If you take the release documentation seriously, you will see that you shouldn't
be building with clang at all, or even a recent version of gcc:

http://llvm.org/docs/HowToReleaseLLVM.html#target-build

Nice hey!

Ciao, Duncan.

>
> ...Takumi
>