[zorg] r245686 - Enable assertions in Polly's LNT build bots

[Tobias Grosser via llvm-commits]

On 08/21/2015 07:27 PM, David Blaikie wrote:
>
>
> On Fri, Aug 21, 2015 at 3:22 AM, Tobias Grosser via llvm-commits <llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>> wrote:
>
>     Author: grosser
>     Date: Fri Aug 21 05:22:37 2015
>     New Revision: 245686
>
>     URL: http://llvm.org/viewvc/llvm-project?rev=245686&view=rev
>     Log:
>     Enable assertions in Polly's LNT build bots
>
>     The common performance configuration is Release+Asserts, hence we want
>     to primarily track this performance.
>
>
> I'm slightly confused here - why would you track the runtime performance of an asserts build? The build system itself prints a pretty stern warning that they can be at least an order of magnitude slower than a release build... (unless you're only interested in the perf of the resulting program, not the compile-time perf)

Thanks David for catching this. I got confused as both autoconf and the LLVM LNT builders use Release+Asserts, but did not check the actual release builds. I reverted this change (and moved another Polly builder to Release only testing).

Thank you,
Tobias