[llvm] [NFC] Switch a number of DenseMaps to SmallDenseMaps for speedup (PR #109417)

LLVM Continuous Integration via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 26 05:41:16 PDT 2024 

llvm-ci wrote:

LLVM Buildbot has detected a new failure on builder `clang-x86_64-debian-fast` running on `gribozavr4` while building `llvm` at step 6 "test-build-unified-tree-check-all".

Full details are available at: https://lab.llvm.org/buildbot/#/builders/56/builds/8371

<details>
<summary>Here is the relevant piece of the build log for the reference</summary>

```
Step 6 (test-build-unified-tree-check-all) failure: test (failure)
...
4.250 [2/20/1155] Linking CXX executable tools/clang/unittests/Serialization/SerializationTests
4.280 [2/19/1156] Linking CXX executable tools/clang/unittests/Index/IndexTests
4.299 [2/18/1157] Linking CXX executable unittests/tools/llvm-exegesis/LLVMExegesisTests
4.309 [2/17/1158] Linking CXX executable unittests/ExecutionEngine/MCJIT/MCJITTests
4.313 [2/16/1159] Linking CXX executable tools/clang/unittests/AST/ASTTests
4.314 [2/15/1160] Linking CXX executable unittests/Target/AMDGPU/AMDGPUTests
5.069 [2/14/1161] Linking CXX executable unittests/MIR/MIRTests
5.081 [2/13/1162] Linking CXX executable tools/clang/unittests/CodeGen/ClangCodeGenTests
5.085 [2/12/1163] Linking CXX executable unittests/Target/TargetMachineCTests
5.116 [2/11/1164] Building CXX object unittests/Analysis/CMakeFiles/AnalysisTests.dir/SparsePropagation.cpp.o
FAILED: unittests/Analysis/CMakeFiles/AnalysisTests.dir/SparsePropagation.cpp.o 
CCACHE_CPP2=yes CCACHE_HASHDIR=yes /usr/bin/ccache /usr/bin/clang++ -DGTEST_HAS_RTTI=0 -DLLVM_BUILD_STATIC -D_DEBUG -D_GLIBCXX_ASSERTIONS -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -I/b/1/clang-x86_64-debian-fast/llvm.obj/unittests/Analysis -I/b/1/clang-x86_64-debian-fast/llvm.src/llvm/unittests/Analysis -I/b/1/clang-x86_64-debian-fast/llvm.obj/include -I/b/1/clang-x86_64-debian-fast/llvm.src/llvm/include -I/b/1/clang-x86_64-debian-fast/llvm.src/third-party/unittest/googletest/include -I/b/1/clang-x86_64-debian-fast/llvm.src/third-party/unittest/googlemock/include -std=c++11 -Wdocumentation -Wno-documentation-deprecated-sync -fPIC -fvisibility-inlines-hidden -Werror=date-time -Werror=unguarded-availability-new -Wall -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wmissing-field-initializers -pedantic -Wno-long-long -Wc++98-compat-extra-semi -Wimplicit-fallthrough -Wcovered-switch-default -Wno-noexcept-type -Wnon-virtual-dtor -Wdelete-non-virtual-dtor -Wsuggest-override -Wstring-conversion -Wmisleading-indentation -Wctad-maybe-unsupported -fdiagnostics-color -ffunction-sections -fdata-sections -O3 -DNDEBUG  -Wno-variadic-macros -Wno-gnu-zero-variadic-macro-arguments -fno-exceptions -funwind-tables -fno-rtti -UNDEBUG -Wno-suggest-override -std=c++17 -MD -MT unittests/Analysis/CMakeFiles/AnalysisTests.dir/SparsePropagation.cpp.o -MF unittests/Analysis/CMakeFiles/AnalysisTests.dir/SparsePropagation.cpp.o.d -o unittests/Analysis/CMakeFiles/AnalysisTests.dir/SparsePropagation.cpp.o -c /b/1/clang-x86_64-debian-fast/llvm.src/llvm/unittests/Analysis/SparsePropagation.cpp
/b/1/clang-x86_64-debian-fast/llvm.src/llvm/unittests/Analysis/SparsePropagation.cpp:143:57: error: non-virtual member function marked 'override' hides virtual member function
      SparseSolver<TestLatticeKey, TestLatticeVal> &SS) override {
                                                        ^
/b/1/clang-x86_64-debian-fast/llvm.src/llvm/include/llvm/Analysis/SparsePropagation.h:90:16: note: hidden overloaded virtual function 'llvm::AbstractLatticeFunction<llvm::PointerIntPair<llvm::Value *, 2, (anonymous namespace)::IPOGrouping, llvm::PointerLikeTypeTraits<llvm::Value *>, llvm::PointerIntPairInfo<llvm::Value *, 2, llvm::PointerLikeTypeTraits<llvm::Value *>>>, (anonymous namespace)::TestLatticeVal>::ComputeInstructionState' declared here: type mismatch at 2nd parameter ('SmallDenseMap<llvm::PointerIntPair<llvm::Value *, 2, (anonymous namespace)::IPOGrouping, llvm::PointerLikeTypeTraits<llvm::Value *>, llvm::PointerIntPairInfo<llvm::Value *, 2, llvm::PointerLikeTypeTraits<llvm::Value *>>>, (anonymous namespace)::TestLatticeVal, 16> &' vs 'DenseMap<(anonymous namespace)::TestLatticeKey, (anonymous namespace)::TestLatticeVal> &' (aka 'DenseMap<PointerIntPair<llvm::Value *, 2, (anonymous namespace)::IPOGrouping>, (anonymous namespace)::TestLatticeVal> &'))
  virtual void ComputeInstructionState(
               ^
/b/1/clang-x86_64-debian-fast/llvm.src/llvm/unittests/Analysis/SparsePropagation.cpp:230:19: error: field type '(anonymous namespace)::TestLatticeFunc' is an abstract class
  TestLatticeFunc Lattice;
                  ^
/b/1/clang-x86_64-debian-fast/llvm.src/llvm/include/llvm/Analysis/SparsePropagation.h:90:16: note: unimplemented pure virtual method 'ComputeInstructionState' in 'TestLatticeFunc'
  virtual void ComputeInstructionState(
               ^
In file included from /b/1/clang-x86_64-debian-fast/llvm.src/llvm/unittests/Analysis/SparsePropagation.cpp:13:
In file included from /b/1/clang-x86_64-debian-fast/llvm.src/third-party/unittest/googletest/include/gtest/gtest.h:65:
In file included from /b/1/clang-x86_64-debian-fast/llvm.src/third-party/unittest/googletest/include/gtest/gtest-death-test.h:43:
In file included from /b/1/clang-x86_64-debian-fast/llvm.src/third-party/unittest/googletest/include/gtest/internal/gtest-death-test-internal.h:47:
In file included from /b/1/clang-x86_64-debian-fast/llvm.src/third-party/unittest/googletest/include/gtest/gtest-matchers.h:49:
In file included from /b/1/clang-x86_64-debian-fast/llvm.src/third-party/unittest/googletest/include/gtest/gtest-printers.h:122:
/b/1/clang-x86_64-debian-fast/llvm.src/third-party/unittest/googletest/include/gtest/internal/gtest-internal.h:457:40: error: cannot initialize return object of type 'testing::Test *' with an rvalue of type 'SparsePropagationTest_MarkBlockExecutable_Test *'
  Test* CreateTest() override { return new TestClass; }
                                       ^~~
/b/1/clang-x86_64-debian-fast/llvm.src/llvm/unittests/Analysis/SparsePropagation.cpp:255:1: note: in instantiation of member function 'testing::internal::TestFactoryImpl<SparsePropagationTest_MarkBlockExecutable_Test>::CreateTest' requested here
TEST_F(SparsePropagationTest, MarkBlockExecutable) {
^
/b/1/clang-x86_64-debian-fast/llvm.src/third-party/unittest/googletest/include/gtest/gtest.h:2208:41: note: expanded from macro 'TEST_F'
#define TEST_F(test_fixture, test_name) GTEST_TEST_F(test_fixture, test_name)
                                        ^
/b/1/clang-x86_64-debian-fast/llvm.src/third-party/unittest/googletest/include/gtest/gtest.h:2205:3: note: expanded from macro 'GTEST_TEST_F'
  GTEST_TEST_(test_fixture, test_name, test_fixture, \
  ^
/b/1/clang-x86_64-debian-fast/llvm.src/third-party/unittest/googletest/include/gtest/internal/gtest-internal.h:1556:15: note: expanded from macro 'GTEST_TEST_'
          new ::testing::internal::TestFactoryImpl<GTEST_TEST_CLASS_NAME_(     \
              ^
/b/1/clang-x86_64-debian-fast/llvm.src/third-party/unittest/googletest/include/gtest/internal/gtest-internal.h:457:40: error: cannot initialize return object of type 'testing::Test *' with an rvalue of type 'SparsePropagationTest_GlobalVariableConstant_Test *'
  Test* CreateTest() override { return new TestClass; }
                                       ^~~
/b/1/clang-x86_64-debian-fast/llvm.src/llvm/unittests/Analysis/SparsePropagation.cpp:291:1: note: in instantiation of member function 'testing::internal::TestFactoryImpl<SparsePropagationTest_GlobalVariableConstant_Test>::CreateTest' requested here

```

</details>

https://github.com/llvm/llvm-project/pull/109417

More information about the llvm-commits mailing list