[llvm] [LV][VPlan] Extract the implementation of transform Recipe to EVLRecipe into a small function. NFC (PR #119510)
LLVM Continuous Integration via llvm-commits
llvm-commits at lists.llvm.org
Sun Dec 22 16:37:42 PST 2024
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `sanitizer-x86_64-linux-android` running on `sanitizer-buildbot-android` while building `llvm` at step 2 "annotate".
Full details are available at: https://lab.llvm.org/buildbot/#/builders/186/builds/5102
<details>
<summary>Here is the relevant piece of the build log for the reference</summary>
```
Step 2 (annotate) failure: 'python ../sanitizer_buildbot/sanitizers/zorg/buildbot/builders/sanitizers/buildbot_selector.py' (failure)
...
[2130/5338] Building CXX object lib/Target/AArch64/MCTargetDesc/CMakeFiles/LLVMAArch64Desc.dir/AArch64MCExpr.cpp.o
[2131/5338] Building CXX object lib/Target/AArch64/MCTargetDesc/CMakeFiles/LLVMAArch64Desc.dir/AArch64MCTargetDesc.cpp.o
[2132/5338] Building CXX object lib/Target/AArch64/MCTargetDesc/CMakeFiles/LLVMAArch64Desc.dir/AArch64MachObjectWriter.cpp.o
[2133/5338] Building CXX object lib/Target/AArch64/MCTargetDesc/CMakeFiles/LLVMAArch64Desc.dir/AArch64TargetStreamer.cpp.o
[2134/5338] Building CXX object lib/Target/AArch64/MCTargetDesc/CMakeFiles/LLVMAArch64Desc.dir/AArch64WinCOFFObjectWriter.cpp.o
[2135/5338] Building CXX object lib/Target/AArch64/MCTargetDesc/CMakeFiles/LLVMAArch64Desc.dir/AArch64WinCOFFStreamer.cpp.o
[2136/5338] Building CXX object lib/Target/AArch64/TargetInfo/CMakeFiles/LLVMAArch64Info.dir/AArch64TargetInfo.cpp.o
[2137/5338] Building CXX object lib/Target/AArch64/Utils/CMakeFiles/LLVMAArch64Utils.dir/AArch64BaseInfo.cpp.o
[2138/5338] Building CXX object lib/Target/AArch64/Utils/CMakeFiles/LLVMAArch64Utils.dir/AArch64SMEAttributes.cpp.o
[2139/5338] Building CXX object lib/Transforms/Vectorize/CMakeFiles/LLVMVectorize.dir/VPlanTransforms.cpp.o
FAILED: lib/Transforms/Vectorize/CMakeFiles/LLVMVectorize.dir/VPlanTransforms.cpp.o
CCACHE_CPP2=yes CCACHE_HASHDIR=yes /usr/bin/ccache /var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm_build0/bin/clang++ -DGTEST_HAS_RTTI=0 -D_DEBUG -D_GLIBCXX_ASSERTIONS -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -I/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm_build64/lib/Transforms/Vectorize -I/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize -I/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm_build64/include -I/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/include -fPIC -fno-semantic-interposition -fvisibility-inlines-hidden -Werror -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 -std=c++17 -fno-exceptions -funwind-tables -fno-rtti -UNDEBUG -MD -MT lib/Transforms/Vectorize/CMakeFiles/LLVMVectorize.dir/VPlanTransforms.cpp.o -MF lib/Transforms/Vectorize/CMakeFiles/LLVMVectorize.dir/VPlanTransforms.cpp.o.d -o lib/Transforms/Vectorize/CMakeFiles/LLVMVectorize.dir/VPlanTransforms.cpp.o -c /var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1495:28: error: variable 'VPID' is used uninitialized whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized]
1495 | else if (auto *CastR = dyn_cast<VPWidenCastRecipe>(CR))
| ^~~~~
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/include/llvm/ADT/TypeSwitch.h:102:22: note: in instantiation of function template specialization 'createEVLRecipe(VPValue *, VPRecipeBase &, VPTypeAnalysis &, VPValue &, VPValue &)::(anonymous class)::operator()<llvm::VPWidenIntrinsicRecipe>' requested here
102 | result.emplace(caseFn(caseValue));
| ^
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/include/llvm/ADT/TypeSwitch.h:46:29: note: in instantiation of function template specialization 'llvm::TypeSwitch<llvm::VPRecipeBase *, llvm::VPRecipeBase *>::Case<llvm::VPWidenIntrinsicRecipe, (lambda at /var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1490:11) &>' requested here
46 | return derived.template Case<CaseT>(caseFn)
| ^
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1489:8: note: in instantiation of function template specialization 'llvm::detail::TypeSwitchBase<llvm::TypeSwitch<llvm::VPRecipeBase *, llvm::VPRecipeBase *>, llvm::VPRecipeBase *>::Case<llvm::VPWidenIntrinsicRecipe, llvm::VPWidenCastRecipe, (lambda at /var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1490:11)>' requested here
1489 | .Case<VPWidenIntrinsicRecipe, VPWidenCastRecipe>(
| ^
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1497:20: note: uninitialized use occurs here
1497 | assert(VPID != Intrinsic::not_intrinsic && "Expected VP intrinsic");
| ^~~~
/usr/include/assert.h:103:27: note: expanded from macro 'assert'
103 | (static_cast <bool> (expr) \
| ^~~~
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1495:18: note: remove the 'if' if its condition is always true
1495 | else if (auto *CastR = dyn_cast<VPWidenCastRecipe>(CR))
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1496 | VPID = VPIntrinsic::getForOpcode(CastR->getOpcode());
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1491:31: note: initialize the variable 'VPID' to silence this warning
1491 | Intrinsic::ID VPID;
| ^
| = 0
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1495:28: error: variable 'VPID' is used uninitialized whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized]
1495 | else if (auto *CastR = dyn_cast<VPWidenCastRecipe>(CR))
| ^~~~~
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/include/llvm/ADT/TypeSwitch.h:102:22: note: in instantiation of function template specialization 'createEVLRecipe(VPValue *, VPRecipeBase &, VPTypeAnalysis &, VPValue &, VPValue &)::(anonymous class)::operator()<llvm::VPWidenCastRecipe>' requested here
102 | result.emplace(caseFn(caseValue));
| ^
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/include/llvm/ADT/TypeSwitch.h:47:19: note: in instantiation of function template specialization 'llvm::TypeSwitch<llvm::VPRecipeBase *, llvm::VPRecipeBase *>::Case<llvm::VPWidenCastRecipe, (lambda at /var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1490:11) &>' requested here
47 | .template Case<CaseT2, CaseTs...>(caseFn);
| ^
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1489:8: note: in instantiation of function template specialization 'llvm::detail::TypeSwitchBase<llvm::TypeSwitch<llvm::VPRecipeBase *, llvm::VPRecipeBase *>, llvm::VPRecipeBase *>::Case<llvm::VPWidenIntrinsicRecipe, llvm::VPWidenCastRecipe, (lambda at /var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1490:11)>' requested here
1489 | .Case<VPWidenIntrinsicRecipe, VPWidenCastRecipe>(
Step 8 (bootstrap clang) failure: bootstrap clang (failure)
...
[2130/5338] Building CXX object lib/Target/AArch64/MCTargetDesc/CMakeFiles/LLVMAArch64Desc.dir/AArch64MCExpr.cpp.o
[2131/5338] Building CXX object lib/Target/AArch64/MCTargetDesc/CMakeFiles/LLVMAArch64Desc.dir/AArch64MCTargetDesc.cpp.o
[2132/5338] Building CXX object lib/Target/AArch64/MCTargetDesc/CMakeFiles/LLVMAArch64Desc.dir/AArch64MachObjectWriter.cpp.o
[2133/5338] Building CXX object lib/Target/AArch64/MCTargetDesc/CMakeFiles/LLVMAArch64Desc.dir/AArch64TargetStreamer.cpp.o
[2134/5338] Building CXX object lib/Target/AArch64/MCTargetDesc/CMakeFiles/LLVMAArch64Desc.dir/AArch64WinCOFFObjectWriter.cpp.o
[2135/5338] Building CXX object lib/Target/AArch64/MCTargetDesc/CMakeFiles/LLVMAArch64Desc.dir/AArch64WinCOFFStreamer.cpp.o
[2136/5338] Building CXX object lib/Target/AArch64/TargetInfo/CMakeFiles/LLVMAArch64Info.dir/AArch64TargetInfo.cpp.o
[2137/5338] Building CXX object lib/Target/AArch64/Utils/CMakeFiles/LLVMAArch64Utils.dir/AArch64BaseInfo.cpp.o
[2138/5338] Building CXX object lib/Target/AArch64/Utils/CMakeFiles/LLVMAArch64Utils.dir/AArch64SMEAttributes.cpp.o
[2139/5338] Building CXX object lib/Transforms/Vectorize/CMakeFiles/LLVMVectorize.dir/VPlanTransforms.cpp.o
FAILED: lib/Transforms/Vectorize/CMakeFiles/LLVMVectorize.dir/VPlanTransforms.cpp.o
CCACHE_CPP2=yes CCACHE_HASHDIR=yes /usr/bin/ccache /var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm_build0/bin/clang++ -DGTEST_HAS_RTTI=0 -D_DEBUG -D_GLIBCXX_ASSERTIONS -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -I/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm_build64/lib/Transforms/Vectorize -I/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize -I/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm_build64/include -I/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/include -fPIC -fno-semantic-interposition -fvisibility-inlines-hidden -Werror -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 -std=c++17 -fno-exceptions -funwind-tables -fno-rtti -UNDEBUG -MD -MT lib/Transforms/Vectorize/CMakeFiles/LLVMVectorize.dir/VPlanTransforms.cpp.o -MF lib/Transforms/Vectorize/CMakeFiles/LLVMVectorize.dir/VPlanTransforms.cpp.o.d -o lib/Transforms/Vectorize/CMakeFiles/LLVMVectorize.dir/VPlanTransforms.cpp.o -c /var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1495:28: error: variable 'VPID' is used uninitialized whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized]
1495 | else if (auto *CastR = dyn_cast<VPWidenCastRecipe>(CR))
| ^~~~~
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/include/llvm/ADT/TypeSwitch.h:102:22: note: in instantiation of function template specialization 'createEVLRecipe(VPValue *, VPRecipeBase &, VPTypeAnalysis &, VPValue &, VPValue &)::(anonymous class)::operator()<llvm::VPWidenIntrinsicRecipe>' requested here
102 | result.emplace(caseFn(caseValue));
| ^
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/include/llvm/ADT/TypeSwitch.h:46:29: note: in instantiation of function template specialization 'llvm::TypeSwitch<llvm::VPRecipeBase *, llvm::VPRecipeBase *>::Case<llvm::VPWidenIntrinsicRecipe, (lambda at /var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1490:11) &>' requested here
46 | return derived.template Case<CaseT>(caseFn)
| ^
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1489:8: note: in instantiation of function template specialization 'llvm::detail::TypeSwitchBase<llvm::TypeSwitch<llvm::VPRecipeBase *, llvm::VPRecipeBase *>, llvm::VPRecipeBase *>::Case<llvm::VPWidenIntrinsicRecipe, llvm::VPWidenCastRecipe, (lambda at /var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1490:11)>' requested here
1489 | .Case<VPWidenIntrinsicRecipe, VPWidenCastRecipe>(
| ^
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1497:20: note: uninitialized use occurs here
1497 | assert(VPID != Intrinsic::not_intrinsic && "Expected VP intrinsic");
| ^~~~
/usr/include/assert.h:103:27: note: expanded from macro 'assert'
103 | (static_cast <bool> (expr) \
| ^~~~
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1495:18: note: remove the 'if' if its condition is always true
1495 | else if (auto *CastR = dyn_cast<VPWidenCastRecipe>(CR))
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1496 | VPID = VPIntrinsic::getForOpcode(CastR->getOpcode());
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1491:31: note: initialize the variable 'VPID' to silence this warning
1491 | Intrinsic::ID VPID;
| ^
| = 0
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1495:28: error: variable 'VPID' is used uninitialized whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized]
1495 | else if (auto *CastR = dyn_cast<VPWidenCastRecipe>(CR))
| ^~~~~
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/include/llvm/ADT/TypeSwitch.h:102:22: note: in instantiation of function template specialization 'createEVLRecipe(VPValue *, VPRecipeBase &, VPTypeAnalysis &, VPValue &, VPValue &)::(anonymous class)::operator()<llvm::VPWidenCastRecipe>' requested here
102 | result.emplace(caseFn(caseValue));
| ^
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/include/llvm/ADT/TypeSwitch.h:47:19: note: in instantiation of function template specialization 'llvm::TypeSwitch<llvm::VPRecipeBase *, llvm::VPRecipeBase *>::Case<llvm::VPWidenCastRecipe, (lambda at /var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1490:11) &>' requested here
47 | .template Case<CaseT2, CaseTs...>(caseFn);
| ^
/var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1489:8: note: in instantiation of function template specialization 'llvm::detail::TypeSwitchBase<llvm::TypeSwitch<llvm::VPRecipeBase *, llvm::VPRecipeBase *>, llvm::VPRecipeBase *>::Case<llvm::VPWidenIntrinsicRecipe, llvm::VPWidenCastRecipe, (lambda at /var/lib/buildbot/sanitizer-buildbot6/sanitizer-x86_64-linux-android/build/llvm-project/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:1490:11)>' requested here
1489 | .Case<VPWidenIntrinsicRecipe, VPWidenCastRecipe>(
```
</details>
https://github.com/llvm/llvm-project/pull/119510
More information about the llvm-commits
mailing list