[PATCH] D63281: [TargetLowering] Add SimplifyMultipleUseDemandedBits

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 15 03:41:53 PDT 2019


nikic added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/TargetLowering.cpp:664
   if (!Op.getNode()->hasOneUse() && !AssumeSingleUse) {
     if (Depth != 0) {
       // If not at the root, Just compute the Known bits to
----------------
Why are there explicit SimplifyMultipleUseDemandedBits() calls for individual opcodes, rather than a general call in the !hasOneUse branch here? This is what we do in InstCombine.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63281





More information about the llvm-commits mailing list