[PATCH] D25847: [ValueTracking] Don't assume we can speculate shifts

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 20 16:20:04 PDT 2016


sanjoy added a comment.

This looks wrong -- shifts should always be safe to speculate.  I wasn't following PR30708 closely, but can you describe why you think shift speculation is the root cause for PR30708?


https://reviews.llvm.org/D25847





More information about the llvm-commits mailing list