[PATCH] D68740: [NFC][CVP] Count all the no-wraps we prooved

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 18 04:47:19 PDT 2019


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

LGTM



================
Comment at: llvm/lib/Transforms/Scalar/CorrelatedValuePropagation.cpp:430
 
+static void overflowingBinaryOperatorFlagsDeduced(Value *V,
+                                                  Instruction::BinaryOps Opcode,
----------------
Nit - have the function name specify the action, so something like:
  static void setDeducedOverflowingFlags(...)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68740





More information about the llvm-commits mailing list