[llvm] [SandboxVectorizer] New class to actually collect and manage seeds (PR #112979)

Mon Oct 21 15:43:58 PDT 2024

llvm-ci wrote:

LLVM Buildbot has detected a new failure on builder `ml-opt-devrel-x86-64` running on `ml-opt-devrel-x86-64-b1` while building `llvm` at step 6 "test-build-unified-tree-check-all".

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

<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)
...
6.506 [2/10/706] Linking CXX executable unittests/Transforms/Utils/UtilsTests
6.952 [2/9/707] Linking CXX executable unittests/MIR/MIRTests
6.982 [2/8/708] Linking CXX executable unittests/CodeGen/CodeGenTests
7.078 [2/7/709] Linking CXX executable unittests/tools/llvm-mca/LLVMMCATests
7.239 [2/6/710] Linking CXX executable unittests/DebugInfo/DWARF/DebugInfoDWARFTests
7.371 [2/5/711] Linking CXX executable unittests/CodeGen/GlobalISel/GlobalISelTests
7.511 [2/4/712] Linking CXX executable unittests/Target/TargetMachineCTests
7.541 [2/3/713] Linking CXX executable unittests/MI/MITests
7.960 [2/2/714] Linking CXX executable unittests/tools/llvm-exegesis/LLVMExegesisTests
12.454 [2/1/715] Building CXX object unittests/Transforms/Vectorize/SandboxVectorizer/CMakeFiles/SandboxVectorizerTests.dir/SeedCollectorTest.cpp.o
FAILED: unittests/Transforms/Vectorize/SandboxVectorizer/CMakeFiles/SandboxVectorizerTests.dir/SeedCollectorTest.cpp.o 
ccache /usr/bin/c++ -DCPUINFO_SUPPORTED_PLATFORM=1 -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/ml-opt-devrel-x86-64-b1/build/unittests/Transforms/Vectorize/SandboxVectorizer -I/b/ml-opt-devrel-x86-64-b1/llvm-project/llvm/unittests/Transforms/Vectorize/SandboxVectorizer -I/var/lib/buildbot/.local/lib/python3.7/site-packages/tensorflow/include -I/b/ml-opt-devrel-x86-64-b1/build/include -I/b/ml-opt-devrel-x86-64-b1/llvm-project/llvm/include -I/b/ml-opt-devrel-x86-64-b1/llvm-project/third-party/unittest/googletest/include -I/b/ml-opt-devrel-x86-64-b1/llvm-project/third-party/unittest/googlemock/include -isystem /tmp/tflitebuild/tensorflow/include -isystem /tmp/tflitebuild/eigen/include/eigen3 -isystem /tmp/tflitebuild/abseil-cpp/include -isystem /tmp/tflitebuild/flatbuffers/include -isystem /tmp/tflitebuild/gemmlowp/include/gemmlowp -isystem /tmp/tflitebuild/ml_dtypes/src/ml_dtypes -isystem /tmp/tflitebuild/ml_dtypes/src/ml_dtypes/ml_dtypes -isystem /tmp/tflitebuild/ruy/include -isystem /tmp/tflitebuild/cpuinfo/include -isystem /tmp/tflitebuild/ARM_NEON_2_x86_SSE/include -fPIC -fno-semantic-interposition -fvisibility-inlines-hidden -Werror=date-time -fno-lifetime-dse -Wall -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wno-missing-field-initializers -pedantic -Wno-long-long -Wimplicit-fallthrough -Wno-uninitialized -Wno-nonnull -Wno-class-memaccess -Wno-redundant-move -Wno-pessimizing-move -Wno-noexcept-type -Wdelete-non-virtual-dtor -Wsuggest-override -Wno-comment -Wno-misleading-indentation -fdiagnostics-color -ffunction-sections -fdata-sections -O3 -DNDEBUG  -Wno-variadic-macros -fno-exceptions -funwind-tables -fno-rtti -UNDEBUG -DEIGEN_NEON_GEBP_NR=4 -DTFL_STATIC_LIBRARY_BUILD -Wno-suggest-override -std=c++17 -MD -MT unittests/Transforms/Vectorize/SandboxVectorizer/CMakeFiles/SandboxVectorizerTests.dir/SeedCollectorTest.cpp.o -MF unittests/Transforms/Vectorize/SandboxVectorizer/CMakeFiles/SandboxVectorizerTests.dir/SeedCollectorTest.cpp.o.d -o unittests/Transforms/Vectorize/SandboxVectorizer/CMakeFiles/SandboxVectorizerTests.dir/SeedCollectorTest.cpp.o -c /b/ml-opt-devrel-x86-64-b1/llvm-project/llvm/unittests/Transforms/Vectorize/SandboxVectorizer/SeedCollectorTest.cpp
In file included from /b/ml-opt-devrel-x86-64-b1/llvm-project/third-party/unittest/googlemock/include/gmock/gmock-matchers.h:274,
                 from /b/ml-opt-devrel-x86-64-b1/llvm-project/llvm/include/llvm/Testing/Support/SupportHelpers.h:17,
                 from /b/ml-opt-devrel-x86-64-b1/llvm-project/llvm/unittests/Transforms/Vectorize/SandboxVectorizer/SeedCollectorTest.cpp:20:
/b/ml-opt-devrel-x86-64-b1/llvm-project/third-party/unittest/googlemock/include/gmock/internal/gmock-internal-utils.h: In instantiation of ‘class testing::internal::StlContainerView<llvm::sandboxir::SeedBundle>’:
/b/ml-opt-devrel-x86-64-b1/llvm-project/third-party/unittest/googlemock/include/gmock/gmock-matchers.h:3786:44:   required from ‘testing::internal::ElementsAreMatcher<MatcherTuple>::operator testing::Matcher<T>() const [with Container = const llvm::sandboxir::SeedBundle&; MatcherTuple = std::tuple<llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*>]’
/b/ml-opt-devrel-x86-64-b1/llvm-project/third-party/unittest/googlemock/include/gmock/gmock-matchers.h:373:12:   required from ‘static testing::Matcher<T> testing::internal::MatcherCastImpl<T, M>::CastImpl(const M&, std::true_type, std::integral_constant<bool, Ignore>) [with bool Ignore = false; T = const llvm::sandboxir::SeedBundle&; M = testing::internal::ElementsAreMatcher<std::tuple<llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*> >; std::true_type = std::integral_constant<bool, true>]’
/b/ml-opt-devrel-x86-64-b1/llvm-project/third-party/unittest/googlemock/include/gmock/gmock-matchers.h:355:20:   required from ‘static testing::Matcher<T> testing::internal::MatcherCastImpl<T, M>::Cast(const M&) [with T = const llvm::sandboxir::SeedBundle&; M = testing::internal::ElementsAreMatcher<std::tuple<llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*> >]’
/b/ml-opt-devrel-x86-64-b1/llvm-project/third-party/unittest/googlemock/include/gmock/gmock-matchers.h:515:47:   required from ‘testing::Matcher<T> testing::MatcherCast(const M&) [with T = const llvm::sandboxir::SeedBundle&; M = testing::internal::ElementsAreMatcher<std::tuple<llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*> >]’
/b/ml-opt-devrel-x86-64-b1/llvm-project/third-party/unittest/googlemock/include/gmock/gmock-matchers.h:522:24:   required from ‘testing::Matcher<T> testing::SafeMatcherCast(const M&) [with T = const llvm::sandboxir::SeedBundle&; M = testing::internal::ElementsAreMatcher<std::tuple<llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*> >]’
/b/ml-opt-devrel-x86-64-b1/llvm-project/third-party/unittest/googlemock/include/gmock/gmock-matchers.h:1591:64:   required from ‘testing::AssertionResult testing::internal::PredicateFormatterFromMatcher<M>::operator()(const char*, const T&) const [with T = llvm::sandboxir::SeedBundle; M = testing::internal::ElementsAreMatcher<std::tuple<llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*, llvm::sandboxir::Instruction*> >]’
/b/ml-opt-devrel-x86-64-b1/llvm-project/llvm/unittests/Transforms/Vectorize/SandboxVectorizer/SeedCollectorTest.cpp:316:3:   required from here
/b/ml-opt-devrel-x86-64-b1/llvm-project/third-party/unittest/googlemock/include/gmock/internal/gmock-internal-utils.h:352:15: error: invalid abstract return type ‘llvm::sandboxir::SeedBundle’
  352 |   static type Copy(const RawContainer& container) { return container; }
      |               ^~~~
In file included from /b/ml-opt-devrel-x86-64-b1/llvm-project/llvm/unittests/Transforms/Vectorize/SandboxVectorizer/SeedCollectorTest.cpp:9:
/b/ml-opt-devrel-x86-64-b1/llvm-project/llvm/include/llvm/Transforms/Vectorize/SandboxVectorizer/SeedCollector.h:27:7: note:   because the following virtual functions are pure within ‘llvm::sandboxir::SeedBundle’:
   27 | class SeedBundle {
      |       ^~~~~~~~~~
/b/ml-opt-devrel-x86-64-b1/llvm-project/llvm/include/llvm/Transforms/Vectorize/SandboxVectorizer/SeedCollector.h:57:16: note:     ‘virtual void llvm::sandboxir::SeedBundle::insert(llvm::sandboxir::Instruction*, llvm::ScalarEvolution&)’
   57 |   virtual void insert(Instruction *I, ScalarEvolution &SE) = 0;
      |                ^~~~~~
ninja: build stopped: subcommand failed.

```

</details>

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