[PATCH] D20176: [MemCpyOpt] Use MaxIntSize in byte instead of bit

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Fri May 13 08:33:05 PDT 2016


joker.eph added inline comments.

================
Comment at: lib/Transforms/Scalar/MemCpyOptimizer.cpp:188
@@ -187,3 +187,3 @@
   unsigned Bytes = unsigned(End-Start);
-  unsigned MaxIntSize = DL.getLargestLegalIntTypeSize();
+  unsigned MaxIntSize = DL.getLargestLegalIntTypeSize() / 8;
   if (MaxIntSize == 0)
----------------
Good catch!
Looks like a misnamed method, should be `getLargestLegalIntTypeSizeInBits()`


http://reviews.llvm.org/D20176





More information about the llvm-commits mailing list