[PATCH] D84971: [ValueTracking] Add ComputeNumSignBits support for llvm.abs intrinsic

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 31 09:02:51 PDT 2020


spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/Analysis/ValueTracking.cpp:3005
+    case Instruction::Call: {
+      if (const IntrinsicInst *II = dyn_cast<IntrinsicInst>(U)) {
+        switch (II->getIntrinsicID()) {
----------------
Could use 'auto *' here per coding standards.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84971/new/

https://reviews.llvm.org/D84971



More information about the llvm-commits mailing list