[PATCH] D11890: Fixed Visual Studio warnings.

Kostya Serebryany via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 12 15:21:51 PDT 2015


kcc added a comment.

I am a big fan of compiler warnings but this one (1 << i) sounds bogus. 
Isn't there a way to represent a size_t value 1 w/o wirting (size_t)1?

(size_t)1 << i
in this case is clearly much better than 
(size_t)(1 << i)


http://reviews.llvm.org/D11890





More information about the llvm-commits mailing list