ToTAndroid64 failure: ld.lld: error: undefined symbol: __arm_tpidr2_save [385758019]

Fixed

Bug

Status Update

No update yet.

Description

ze...@google.com

created issue #1

Dec 23, 2024 08:43PM

Start from: https://ci.chromium.org/ui/p/chromium/builders/ci/ToTAndroid64/23273/overview Log:

[124576/130655] SOLINK ./libcapture_unittests__library.so
python3 ../../build/toolchain/gcc_solink_wrapper.py --readelf=../../third_party/llvm-build/Release+Asserts/bin/llvm-readelf --nm=../../third_p...(too long)
ld.lld: error: undefined symbol: __arm_tpidr2_save
>>> referenced by rotate_sme.cc:32 (../../third_party/libyuv/source/rotate_sme.cc:32)
>>>               libyuv_sme/rotate_sme.o:(TransposeWxH_SME) in archive obj/third_party/libyuv/libyuv_sme.a
>>> referenced by rotate_sme.cc:101 (../../third_party/libyuv/source/rotate_sme.cc:101)
>>>               libyuv_sme/rotate_sme.o:(TransposeUVWxH_SME) in archive obj/third_party/libyuv/libyuv_sme.a
clang++: error: linker command failed with exit code 1 (use -v to see invocation)
exit=1
error: 1 steps failed: exit=1

Comments

ze...@google.com <ze...@google.com> #2Dec 23, 2024 08:58PM

The llvm commits between good and bad builds:

811f2a652b62 [Compiler-rt] Add AArch64 routines for __arm_agnostic("sme_za_state") (#120059)
8177bf5022c6 [Hexagon] Only handle simple types memory accesses (#120654)
56ffcd4e0185 [mlir] Fix integration tests after #120580 (part 2) (#120751)
953b07febca4 [mlir] AMDGPUToROCDL: RawBufferOpLowering fixes (#120642)
5845298f9439 [LoopVectorize] Teach some X86 cost model tests to use new vplan costs (#120738)
70eac255b8c0 [VectorCombine] Add fp cast handling for shuffletoidentity (#120641)
b87a5fb9fd8d [VectorCombine] Add "VC: Visiting" debug message to help the log show the instruction folding order.
2d5dc5c20853 [clang][dataflow] Fix a missing break from a switch case -Wimplicit-fallthrough (#120739)
acfd26a93be3 [SCEV] Fix exit condition for recursive loop guard collection (#120442)
9e333872199b [clang analyzer]consume `llvm::Error` (#120597)
fa9cef50b1af Only guard loop metadata that has non-debug info in it (#118825)
54665f525269 [gn build] Port 54309b1c2f7a
54309b1c2f7a [clang][dataflow] Add matchers for smart pointer accessors to be cached (#120102)
42873e0cf107 [bazel] port eb6c4197d5263ed2e086925b2b2f032a19442d2b
5f0db7c11264 [VectorCombine] Add "VECTORCOMBINE on <FUNCTION_NAME>" title debug message to help finding vectorcombine stages in the debug log
c5434804eeea [VectorCombine] foldInsExtVectorToShuffle - add debug message for match + cost-comparison
a9034d0b7ff3 [clang-tidy][docs] improve documentation on cppcoreguidelines-narrowing-conversions (#111510) (#118209)
4eba40c604c7 [ConstraintElim] Remove dead code. NFC. (#118983)
0dc086a787a4 [mlir] Fix integration tests after #120580 (#120729)
e11d49cbf5a2 [BOLT][AArch64] Adds tls relocations support (#117465)
1738b75b6154 [LLVM] Update InstCombine maintainers (#120408)
eb6c4197d526 [mlir][CF] Split `cf-to-llvm` from `func-to-llvm` (#120580)
cf7b3f8d827a Fix double-quotes in diagnostic when attempting to access a ext_vector of bools (#118186)
451a80ccc034 [docs] Mention ffmpeg and dav1d in llvm-test-suite (#120570)
e4db3f0d9768 [llvm] Bail out when meeting pointer with negative offset in approximated mode instead of … (#120424)
000febd02906 [lldb][test] Add test-coverage for DW_AT_APPLE_objc_complete_type parsing (#120279)
d7ddc976d544 [MachinePipeliner] Remove unused private field MF
8dc23efbe6c5 [NFC][AArch64][SVE] Rename variables in partial reduction lowering functions (#120589)
385b144c9477 [lldb][Linux] Moving generic APIs from HostInfoLinux to HostInfoPosix (#119694)
9a1837f9b0d3 [lldb][AIX] Introducing _ALL_SOURCE macro into driver CMakeLists (#120607)
84f0098ad103 [bazel] port b03a09e74fa38eceddbc314c4f896a935224f453
6fd267d79b9b [lldb][DWARFASTParserClang][NFC] Remove unused parameter to CompleteRecordType (#120456)
cf0bc8d0321a [lldb][AIX] Adding AIX version of ptrace64 (#120390)
c2bd5c25b363 [AArch64] Avoid GPR trip when moving truncated i32 vector elements (#114541)
919aead1db64 [Flang OpenMP] Add LLVM translation support for UNTIED in Task (#115283)
42bc7bf40a13 [gn build] Port d8a5fae6913a
708e1437ff82 [gn build] Port 0b5b09b67c57
ff93ca7d6c48 [VectorCombine] Combine scalar fneg with insert/extract to vector fneg when length is different (#120461)
611401c11594 [CostModel][X86] getShuffleCost - use processShuffleMasks to split SK_PermuteTwoSrc shuffles to legal types (#120599)
1e18815fdc13 [MLIR] fix shape.broadcast canonicalize with all empty shape operands (#118941)
d66f653c8db9 [MachinePipeliner] Skip reserved registers when computing register pressure (#120694)
2fa2c2197ddb [llvm][docs] MemTagSanitizer is only supported on AArch64 Android (#120545)
0b5b09b67c57 [NFC] Move DroppedVariableStats to its own file (#120711)
2405c5fb3ed4 [llvm-readelf] Update outdated URL (#120498)
091448e3c17b Revert "[CostModel][X86] getShuffleCost - use processShuffleMasks to split SK_PermuteTwoSrc shuffles to legal types" (#120707)
b03a09e74fa3 [mlir] Fix integration tests after #120548 (#120706)
69ebac7ad6ae [win/asan] Don't intercept memset etc. in ntdll (#120397)
e10cb443a1ba Revert "[compiler-rt] Add weak defs for .*contiguous_container.* functions (#120376)"
81e63f9e0c4b [CostModel][X86] getShuffleCost - use processShuffleMasks to split SK_PermuteTwoSrc shuffles to legal types (#120599)
fe85c71a7b55 [bazel] port 53d080c5b5dfbb46eb81d189736864f5b6196492
d8a5fae6913a [MC][Mips] Add MipsWinCOFFObjectWriter/MipsWinCOFFStreamer (#114611)
d0b7633d7ad5 [mlir] [doc] fix typos in documentation (#120179)
53d080c5b5df [mlir][Arith] Remove `arith-to-llvm` from `func-to-llvm` (#120548)
701f2409befa [mlir] fix crash when scf utils work on llvm.func (#120688)
44726489988a [ARM] Expand bf16 expanding/rounding fp loads/stores
4111841f88ee [BOLT] Correctly print preferred disassembly for annotated instructions (#120564)
4096dd60668c [LLVM] Update LTO maintainer (#120575)
a16cb7ebea79 [LLVM] Update Sparc maintainer (#120430)
a6d26c56ff06 [clang] Fix dangling false positives for conditional operators. (#120233)
93743ee56669 Revert "[Clang] Re-write codegen for atomic_test_and_set and atomic_clear (#120449)"
59c7d6f965af [Xtensa,test] Remove failing RUN line from #119639
e30308196901 [X86] Remove redundant initialize*Pass in ctor
ecd59f802f94 [SelectionDAG] Use SmallVectorImpl& to avoid repeating SmallVector size. NFC
c7606710f93c [AMDGPU] Update base addr of dyn alloca considering GrowingUp stack (#119822)
a73ca291547c [compiler-rt] Add weak defs for .*contiguous_container.* functions (#120376)

Nothing stands out.

ze...@google.com <ze...@google.com> #3Dec 24, 2024 06:38AM

Bisection points to

811f2a652b6232f203e51a17d553e268fae3a29c is the first bad commit
commit 811f2a652b6232f203e51a17d553e268fae3a29c
Author: Sander de Smalen <sander.desmalen@arm.com>
Date:   Fri Dec 20 15:49:33 2024 +0000

    [Compiler-rt] Add AArch64 routines for __arm_agnostic("sme_za_state") (#120059)

    The specification of these routines can be found here:

    https://github.com/ARM-software/abi-aa/blob/main/aapcs64/aapcs64.rst#sme-support-routines

 compiler-rt/cmake/builtin-config-ix.cmake         |   3 +-
 compiler-rt/lib/builtins/aarch64/sme-abi-assert.c |   1 +
 compiler-rt/lib/builtins/aarch64/sme-abi.S        | 167 +++++++++++++++++++++-
 3 files changed, 169 insertions(+), 2 deletions(-)
bisect found first bad commit

th...@chromium.org <th...@chromium.org> #4Dec 24, 2024 06:59PM

(Left a comment on https://github.com/llvm/llvm-project/pull/120059#issuecomment-2561351574)

ze...@google.com <ze...@google.com> #5Jan 9, 2025 10:25PM

Marked as fixed.

The ToTAnroid bot is green now: https://ci.chromium.org/ui/p/chromium/builders/ci/ToTAndroid64.

Issue 385758019

Description

Issue summary

Comments

ze...@google.com <ze...@google.com> #2Dec 23, 2024 08:58PM

ze...@google.com <ze...@google.com> #3Dec 24, 2024 06:38AM

th...@chromium.org <th...@chromium.org> #4Dec 24, 2024 06:59PM

ze...@google.com <ze...@google.com> #5Jan 9, 2025 10:25PM

Add comment

Issue metadata