[PATCH] D11890: Fixed Visual Studio warnings.

Kostya Serebryany via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 10 13:38:14 PDT 2015


kcc added a subscriber: kcc.

================
Comment at: lib/Transforms/Instrumentation/ThreadSanitizer.cpp:145
@@ -144,3 +144,3 @@
   for (size_t i = 0; i < kNumberOfAccessSizes; ++i) {
-    const size_t ByteSize = 1 << i;
+    const size_t ByteSize = size_t(1 << i);
     const size_t BitSize = ByteSize * 8;
----------------
Wouldn't it be better (more readable to write like this? (Here and below)
  .. = 1UL << i;


http://reviews.llvm.org/D11890





More information about the llvm-commits mailing list