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

Michael Kuperstein via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 20 15:59:31 PDT 2016


mkuper created this revision.
mkuper added reviewers: davide, hfinkel, majnemer.
mkuper added a subscriber: llvm-commits.

Shifts are only isSafeToSpeculativelyExecute if we know the amount we're shifting by is smaller than the bitwidth of the shifted value.

This fixes PR30708


https://reviews.llvm.org/D25847

Files:
  lib/Analysis/ValueTracking.cpp
  test/Transforms/SimplifyCFG/speculat-shift.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D25847.75362.patch
Type: text/x-patch
Size: 4944 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161020/0033df64/attachment-0001.bin>


More information about the llvm-commits mailing list