[llvm-commits] [PATCH] Multidimensional Array Index Delinearization Analysis

[Tobias Grosser]

On 09/26/2012 04:34 PM, Hal Finkel wrote:
> On Wed, 26 Sep 2012 10:10:28 +0200
> Tobias Grosser <tobias at grosser.es> wrote:
>
>> On 09/26/2012 12:02 AM, Hal Finkel wrote:
>>> I've attached an updated version of the pass. This version works
>>> better (several bugs have been fixed), and also includes a working
>>> bounds analysis.
>>
>> Hi Hal,
>>
>> I tested this one again and found it has still problems. himeno_2.ll
>> gives the expected output, himeno_1.ll does not.
>
> You need to run LICM first in order for case 1 to work correctly. So
> running opt -analyze -licm -delinearize and things are fine.

Would it be difficult to avoid this? The original test case had 
branches/loops that blocked LICM. (If needed, I could provide a test 
case where -licm does not perform a transformation).

Cheers
Tobi