[llvm] r364624 - Silence gcc warning in testcase [NFC]

Mikael Holmen via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 27 23:45:20 PDT 2019


Author: uabelho
Date: Thu Jun 27 23:45:20 2019
New Revision: 364624

URL: http://llvm.org/viewvc/llvm-project?rev=364624&view=rev
Log:
Silence gcc warning in testcase [NFC]

Without the fix gcc (7.4.0) complains with

../unittests/ADT/APIntTest.cpp: In member function 'virtual void {anonymous}::APIntTest_MultiplicativeInverseExaustive_Test::TestBody()':
../unittests/ADT/APIntTest.cpp:2510:36: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
     for (unsigned Value = 0; Value < (1 << BitWidth); ++Value) {
                              ~~~~~~^~~~~~~~~~~~~~~~~

Modified:
    llvm/trunk/unittests/ADT/APIntTest.cpp

Modified: llvm/trunk/unittests/ADT/APIntTest.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/ADT/APIntTest.cpp?rev=364624&r1=364623&r2=364624&view=diff
==============================================================================
--- llvm/trunk/unittests/ADT/APIntTest.cpp (original)
+++ llvm/trunk/unittests/ADT/APIntTest.cpp Thu Jun 27 23:45:20 2019
@@ -2507,7 +2507,7 @@ TEST(APIntTest, SolveQuadraticEquationWr
 
 TEST(APIntTest, MultiplicativeInverseExaustive) {
   for (unsigned BitWidth = 1; BitWidth <= 16; ++BitWidth) {
-    for (unsigned Value = 0; Value < (1 << BitWidth); ++Value) {
+    for (unsigned Value = 0; Value < (1u << BitWidth); ++Value) {
       APInt V = APInt(BitWidth, Value);
       APInt MulInv =
           V.zext(BitWidth + 1)




More information about the llvm-commits mailing list