[LLVMdev] DAGCompiler::MergeConsecutiveStores Question

[Nadav Rotem]

Hi David, 

You are right. This check is overly restrictive. We can replace this check with code that uses the alignment of the first store. 

Thanks,
Nadav

On Nov 22, 2013, at 9:31 AM, dag at cray.com wrote:

> In DAGCombiner::MergeConsecutiveStores, there is this check:
> 
>     if (Index->getAlignment() != St->getAlignment())
>       break;
> 
> Apparently this check ensures that all of the stores have the same
> alignment.  Why is that necessary?  This seems very overly restrictive
> to me.
> 
>                            -David
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev