[PATCH] D33166: [ValueTracking] Replace all uses of ComputeSignBit with computeKnownBits.

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstCombineInternal.h:496
                         unsigned Depth, Instruction *CxtI) const {
     return llvm::computeKnownBits(V, Known, DL, Depth, &AC, CxtI, &DT);
   }
----------------
Why on earth are we returning a void here?


https://reviews.llvm.org/D33166