Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Xtensa] Add XtensaDesc dependency to LLVMXtensaDisassembler #126904

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

e-kud
Copy link
Contributor

@e-kud e-kud commented Feb 12, 2025

There is an undefined reference to checkRegister in shared lib builds

Suspecting #124656

There is an undefined reference to checkRegister in shared builds

Suspecting llvm#124656
@arsenm arsenm merged commit 99816a5 into llvm:main Feb 12, 2025
10 checks passed
@llvmbot
Copy link
Member

llvmbot commented Feb 12, 2025

@llvm/pr-subscribers-backend-xtensa

Author: Evgenii Kudriashov (e-kud)

Changes

There is an undefined reference to checkRegister in shared lib builds

Suspecting #124656


Full diff: https://github.com/llvm/llvm-project/pull/126904.diff

1 Files Affected:

  • (modified) llvm/lib/Target/Xtensa/Disassembler/CMakeLists.txt (+1)
diff --git a/llvm/lib/Target/Xtensa/Disassembler/CMakeLists.txt b/llvm/lib/Target/Xtensa/Disassembler/CMakeLists.txt
index 43f235b7cd31d..319d912e5561b 100644
--- a/llvm/lib/Target/Xtensa/Disassembler/CMakeLists.txt
+++ b/llvm/lib/Target/Xtensa/Disassembler/CMakeLists.txt
@@ -4,6 +4,7 @@ add_llvm_component_library(LLVMXtensaDisassembler
   LINK_COMPONENTS
   MCDisassembler
   Support
+  XtensaDesc
   XtensaInfo
 
   ADD_TO_COMPONENT

@llvm-ci
Copy link
Collaborator

llvm-ci commented Feb 13, 2025

LLVM Buildbot has detected a new failure on builder clang-solaris11-sparcv9 running on solaris11-sparcv9 while building llvm at step 5 "ninja check 1".

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

Here is the relevant piece of the build log for the reference
Step 5 (ninja check 1) failure: stage 1 checked (failure)
******************** TEST 'AddressSanitizer-Unit :: ./Asan-sparc-calls-Dynamic-Test/17/47' FAILED ********************
Script(shard):
--
GTEST_OUTPUT=json:/opt/llvm-buildbot/home/solaris11-sparcv9/clang-solaris11-sparcv9/stage1/projects/compiler-rt/lib/asan/tests/SPARCSunOSDynamicConfig/./Asan-sparc-calls-Dynamic-Test-AddressSanitizer-Unit-3493-17-47.json GTEST_SHUFFLE=0 GTEST_TOTAL_SHARDS=47 GTEST_SHARD_INDEX=17 /opt/llvm-buildbot/home/solaris11-sparcv9/clang-solaris11-sparcv9/stage1/projects/compiler-rt/lib/asan/tests/SPARCSunOSDynamicConfig/./Asan-sparc-calls-Dynamic-Test
--

Script:
--
/opt/llvm-buildbot/home/solaris11-sparcv9/clang-solaris11-sparcv9/stage1/projects/compiler-rt/lib/asan/tests/SPARCSunOSDynamicConfig/./Asan-sparc-calls-Dynamic-Test --gtest_filter=AddressSanitizer.OOB_char
--
/opt/llvm-buildbot/home/solaris11-sparcv9/clang-solaris11-sparcv9/llvm/compiler-rt/lib/asan/tests/asan_oob_test.cpp:62: Failure
Death test: oob_test<T>(size, i)
    Result: died but not with expected error.
  Expected: contains regular expression "is located.*5 byte.*before"
Actual msg:
[  DEATH   ] =================================================================
[  DEATH   ] ==3670==ERROR: AddressSanitizer: heap-buffer-overflow on address 0xfc00140b at pc 0xfe4f68d8 bp 0xffbfe3e0 sp 0xffbfe3dc
[  DEATH   ] WRITE of size 1 at 0xfc00140b thread T0
[  DEATH   ] AddressSanitizer:DEADLYSIGNAL
[  DEATH   ] AddressSanitizer: nested bug in the same thread, aborting.
[  DEATH   ] 


/opt/llvm-buildbot/home/solaris11-sparcv9/clang-solaris11-sparcv9/llvm/compiler-rt/lib/asan/tests/asan_oob_test.cpp:62
Death test: oob_test<T>(size, i)
    Result: died but not with expected error.
  Expected: contains regular expression "is located.*5 byte.*before"
Actual msg:
[  DEATH   ] =================================================================
[  DEATH   ] ==3670==ERROR: AddressSanitizer: heap-buffer-overflow on address 0xfc00140b at pc 0xfe4f68d8 bp 0xffbfe3e0 sp 0xffbfe3dc
[  DEATH   ] WRITE of size 1 at 0xfc00140b thread T0
[  DEATH   ] AddressSanitizer:DEADLYSIGNAL
[  DEATH   ] AddressSanitizer: nested bug in the same thread, aborting.
[  DEATH   ] 



********************


@llvm-ci
Copy link
Collaborator

llvm-ci commented Feb 13, 2025

LLVM Buildbot has detected a new failure on builder lld-x86_64-win running on as-worker-93 while building llvm at step 7 "test-build-unified-tree-check-all".

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

Here is the relevant piece of the build log for the reference
Step 7 (test-build-unified-tree-check-all) failure: test (failure)
******************** TEST 'LLVM-Unit :: Support/./SupportTests.exe/38/87' FAILED ********************
Script(shard):
--
GTEST_OUTPUT=json:C:\a\lld-x86_64-win\build\unittests\Support\.\SupportTests.exe-LLVM-Unit-8864-38-87.json GTEST_SHUFFLE=0 GTEST_TOTAL_SHARDS=87 GTEST_SHARD_INDEX=38 C:\a\lld-x86_64-win\build\unittests\Support\.\SupportTests.exe
--

Script:
--
C:\a\lld-x86_64-win\build\unittests\Support\.\SupportTests.exe --gtest_filter=ProgramEnvTest.CreateProcessLongPath
--
C:\a\lld-x86_64-win\llvm-project\llvm\unittests\Support\ProgramTest.cpp(160): error: Expected equality of these values:
  0
  RC
    Which is: -2

C:\a\lld-x86_64-win\llvm-project\llvm\unittests\Support\ProgramTest.cpp(163): error: fs::remove(Twine(LongPath)): did not return errc::success.
error number: 13
error message: permission denied



C:\a\lld-x86_64-win\llvm-project\llvm\unittests\Support\ProgramTest.cpp:160
Expected equality of these values:
  0
  RC
    Which is: -2

C:\a\lld-x86_64-win\llvm-project\llvm\unittests\Support\ProgramTest.cpp:163
fs::remove(Twine(LongPath)): did not return errc::success.
error number: 13
error message: permission denied




********************


flovent pushed a commit to flovent/llvm-project that referenced this pull request Feb 13, 2025
…6904)

There is an undefined reference to checkRegister in shared lib builds

Suspecting llvm#124656
joaosaffran pushed a commit to joaosaffran/llvm-project that referenced this pull request Feb 14, 2025
…6904)

There is an undefined reference to checkRegister in shared lib builds

Suspecting llvm#124656
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants