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

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 25 06:08:19 PDT 2017


ABataev added inline comments.


================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:2972
+  if (unsigned Alignment = PStore->getAlignment())
+    SI->setAlignment(Alignment);
 
----------------
efriedma wrote:
> What happens if PStore and QStore have different alignment?
Generally speaking, they must be the same.


https://reviews.llvm.org/D36841





More information about the llvm-commits mailing list