[PATCH] D36841: [SimplifyCFG] Fix for PR34219: Preserve alignment after merging conditional stores.

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

> Why? If have 2 stores with different alignments, we know that it is safe to use both of them. We should use the biggest one.

If we could prove both stores execute, we could use biggest one.  In this case, though, we only know that one of the stores executes (assuming I'm reading this code correctly).  And we don't know it's safe to take the alignment from a store that doesn't execute.  So we have to use the smaller alignment.


https://reviews.llvm.org/D36841