[llvm] r317578 - Silence MSVC error C2398

Kristof Beyls via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 7 06:37:01 PST 2017


Author: kbeyls
Date: Tue Nov  7 06:37:01 2017
New Revision: 317578

URL: http://llvm.org/viewvc/llvm-project?rev=317578&view=rev
Log:
Silence MSVC error C2398

Reported by http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-win/builds/6000/steps/build-unified-tree/logs/stdio
The error messages were all similar to:
llvm\unittests\CodeGen\GlobalISel\LegalizerInfoTest.cpp(54): error C2398: Element '1': conversion from '' to 'unsigned int' requires a narrowing conversion

Modified:
    llvm/trunk/unittests/CodeGen/GlobalISel/LegalizerInfoTest.cpp

Modified: llvm/trunk/unittests/CodeGen/GlobalISel/LegalizerInfoTest.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/CodeGen/GlobalISel/LegalizerInfoTest.cpp?rev=317578&r1=317577&r2=317578&view=diff
==============================================================================
--- llvm/trunk/unittests/CodeGen/GlobalISel/LegalizerInfoTest.cpp (original)
+++ llvm/trunk/unittests/CodeGen/GlobalISel/LegalizerInfoTest.cpp Tue Nov  7 06:37:01 2017
@@ -49,7 +49,7 @@ TEST(LegalizerInfoTest, ScalarRISC) {
   using namespace TargetOpcode;
   LegalizerInfo L;
   // Typical RISCy set of operations based on AArch64.
-  for (auto Op : {G_ADD, G_SUB}) {
+  for (unsigned Op : {G_ADD, G_SUB}) {
     for (unsigned Size : {32, 64})
       L.setAction({Op, 0, LLT::scalar(Size)}, LegalizerInfo::Legal);
     L.setLegalizeScalarToDifferentSizeStrategy(
@@ -58,7 +58,7 @@ TEST(LegalizerInfoTest, ScalarRISC) {
 
   L.computeTables();
 
-  for (auto &opcode : {G_ADD, G_SUB}) {
+  for (unsigned opcode : {G_ADD, G_SUB}) {
     // Check we infer the correct types and actually do what we're told.
     ASSERT_EQ(L.getAction({opcode, LLT::scalar(8)}),
               std::make_pair(LegalizerInfo::WidenScalar, LLT::scalar(32)));




More information about the llvm-commits mailing list