[llvm-commits] [llvm] r58718 - /llvm/trunk/lib/Transforms/Scalar/ScalarReplAggregates.cpp

[Chris Lattner]

On Nov 4, 2008, at 1:27 PM, Matthijs Kooijman - Inter-Actief wrote:

> Hi Johannes,
>
>> Allow SROA of vectors.  Removing this caused a
>> huge performance regression in something we care
>> about.  This may not be final fix.
> Good to see this fix. I had intended to work out something like  
> this, but
> never found the time.
>
> However, you're saying "Removing this", but AFAIK SROA of vectors  
> never
> happened? I recently touched this code, but only made the "don't touch
> vectors" limitation slightly less strict, I didn't add it AFAICS?

Hi Matthijs,

I think the right fix for this problem is to make it so that GEP can't  
index into a vector element.

-Chris