[PATCH] D46380: [InstSimplify] Adding safety checks for 'shl', 'ashr' and 'lshr'

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a reviewer: lebedev.ri.
spatel added a comment.

I see the motivation better now, but I'm still worried about using ValueTracking.

Can you collect some stats as was done in https://reviews.llvm.org/D47891? I'm curious to know:

1. How often do we call ValueTracking?
2. How often does this transform occur?

Reasonable benchmarks are the same as in the other patch: test-suite or compiling clang/llvm.


Repository:
  rL LLVM

https://reviews.llvm.org/D46380