[llvm-commits] [llvm] r164641 - in /llvm/trunk: lib/Transforms/Scalar/SROA.cpp test/Transforms/SROA/basictest.ll

[Nick Lewycky]

Chandler Carruth wrote:
> Ok, across a few commits I've significantly improved the state of
> setting up alignment properly for loads and stores. There is still work
> to be done here though as there are more places where we fail to track
> it properly. I'll continue looking at this, and especially the
> stable_sort failure as I can.

Thanks Chandler!

When I was looking at the alignment issue with the memcpy, I noticed one 
other thing: load/store ordering and synchscope. There are certainly 
some checks for isSimple in a few places, but it wasn't obvious to me 
that we leave all !isSimple loads and stores alone, and I'm pretty sure 
we drop those properties along with alignment.

Nick