SLP/Loop vectorizer pass ordering

[Michael Zolotukhin]

On Oct 29, 2014, at 11:16 PM, Joerg Sonnenberger <joerg at britannica.bec.de> wrote:
> 
> On Wed, Oct 29, 2014 at 04:12:41PM -0700, Michael Zolotukhin wrote:
>> Okay, seemingly it’s just a bug in the vectorizer: it updates DT incorrectly. Specifically, this line is incorrect:
>>  DT->changeImmediateDominator(LoopExitBlock, LoopMiddleBlock);
>> 
>> The correct dominator for LoopExitBlock in this case would be LoopBypassBlocks[0] (not LoopMiddleBlock). I’ll send a patch soon.
> 
> Could this be the problem behind 21302 too?
I just checked the test from r21302, and it fails even with this patch.

Michael

> 
> Joerg
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits