[PATCH] D154953: [InstCombine] Remove the reminder loop if we know the mask is always true

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 11 06:45:31 PDT 2023


nikic requested changes to this revision.
nikic added inline comments.
This revision now requires changes to proceed.
Herald added a subscriber: StephenFan.


================
Comment at: llvm/include/llvm/Transforms/InstCombine/InstCombiner.h:492
+  bool isVScaleKnownToBeAPowerOfTwo() {
+    return TTI.isVScaleKnownToBeAPowerOfTwo();
+  }
----------------
This needs to be either a data layout property or a function attribute. Though it would probably best to change LangRef to require that vscale is //always// a power of two -- I think consensus has shifted towards non-pow2 vscales not being necessary.


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

https://reviews.llvm.org/D154953



More information about the llvm-commits mailing list