[llvm-commits] [test-suite] r60375 - /test-suite/trunk/SingleSource/Benchmarks/Misc-C++/llloops.cpp

[Chris Lattner]

On Dec 1, 2008, at 2:37 PM, Dale Johannesen wrote:

>
> On Dec 1, 2008, at 2:23 PMPST, Chris Lattner wrote:
>
>> Author: lattner
>> Date: Mon Dec  1 16:23:16 2008
>> New Revision: 60375
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=60375&view=rev
>> Log:
>> Remove this benchmark, it is using undefined behavior (accessing
>> at least the 'u' array beyond its bounds in 'init') and is too
>> brain twisting to fix. Owen, if you really really really want this
>> benchmark, feel free to fix it and recommit.
>
> Livermore Loops is an old but well-known benchmark, well worth
> fixing.  It is brain twisting partly because it was translated from
> Fortran, and partly because the Fortran was.:)
>
> It appears the code in "init" is trying to initialize the entire
> Arrays struct, so bolting on a union with double[totalsize] ought to
> work.

Sure, patches welcome.  :)  We do have other versions of livermore  
loops elsewhere in the testsuite.

-Chris