[PATCH] D86697: [SVE][CodeGen] Fix TypeSize/ElementCount related warnings in sve-split-load.ll

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 31 15:03:37 PDT 2020


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

LGTM with one minor comment.



================
Comment at: llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp:767
     Chain = Ch;
-  } else if (SrcWidth & (SrcWidth - 1)) {
+  } else if (!isPowerOf2_32(SrcWidth.getKnownMinSize())) {
     // If not loading a power-of-2 number of bits, expand as two loads.
----------------
isPowerOf2_64() so you don't have implicit truncation.


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

https://reviews.llvm.org/D86697



More information about the llvm-commits mailing list