Skip to content

[lldb] Fix TestFrameLanguageCommands.py build #141829

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

Merged

Conversation

kastiglione
Copy link
Contributor

@kastiglione kastiglione commented May 28, 2025

The use of -lobjc resulted in this test failing to build on Linux. The flag is not necessary, this removes the flag with the expectation it will fix the test on Linux.

@kastiglione kastiglione requested a review from JDevlieghere as a code owner May 28, 2025 19:24
@llvmbot llvmbot added the lldb label May 28, 2025
@llvmbot
Copy link
Member

llvmbot commented May 28, 2025

@llvm/pr-subscribers-lldb

Author: Dave Lee (kastiglione)

Changes

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

1 Files Affected:

  • (modified) lldb/test/API/commands/command/language/Makefile (-1)
diff --git a/lldb/test/API/commands/command/language/Makefile b/lldb/test/API/commands/command/language/Makefile
index 2d5049417ee70..ce845d59ac035 100644
--- a/lldb/test/API/commands/command/language/Makefile
+++ b/lldb/test/API/commands/command/language/Makefile
@@ -1,4 +1,3 @@
 OBJCXX_SOURCES := main.mm
 CXX_SOURCES := lib.cpp
-LD_EXTRAS := -lobjc
 include Makefile.rules

@kastiglione kastiglione merged commit e653dc9 into llvm:main May 28, 2025
8 of 11 checks passed
@kastiglione kastiglione deleted the lldb-Fix-TestFrameLanguageCommands.py-build branch May 28, 2025 19:25
@kastiglione
Copy link
Contributor Author

See #136766

kastiglione added a commit to swiftlang/llvm-project that referenced this pull request May 29, 2025
The use of `-lobjc` resulted in this test failing to build on Linux. The
flag is not necessary, this removes the flag with the expectation it
will fix the test on Linux.
(cherry-picked from commit e653dc9)
google-yfyang pushed a commit to google-yfyang/llvm-project that referenced this pull request May 29, 2025
The use of `-lobjc` resulted in this test failing to build on Linux. The
flag is not necessary, this removes the flag with the expectation it
will fix the test on Linux.
adrian-prantl pushed a commit to swiftlang/llvm-project that referenced this pull request Jun 2, 2025
…rent frame (llvm#136766) (#10755)

* [lldb] Expose language plugin commands based based on language of current frame (llvm#136766)

Use the current frame's language to lookup commands provided by language plugins.

This means commands like `language {objc,cplusplus} <command>` can be used directly, without using the `language <lang>` prefix.

For example, when stopped on a C++ frame, `demangle _Z1fv` will run `language cplusplus demangle _Z1fv`.

rdar://149882520
(cherry picked from commit 7ed185a)

* [lldb] Fix typo in tagged-pointer syntax string (NFC) (llvm#137069)


(cherry-picked from commit cef9ed5)

* [lldb] Run TestFrameLanguageCommands.py only on Darwin (llvm#141866)

Tests using ObjC do not readily run on Linux.
(cherry-picked from commit 1d48e55)

* [lldb] Fix TestFrameLanguageCommands.py build (llvm#141829)

The use of `-lobjc` resulted in this test failing to build on Linux. The
flag is not necessary, this removes the flag with the expectation it
will fix the test on Linux.
(cherry-picked from commit e653dc9)
sivan-shani pushed a commit to sivan-shani/llvm-project that referenced this pull request Jun 3, 2025
The use of `-lobjc` resulted in this test failing to build on Linux. The
flag is not necessary, this removes the flag with the expectation it
will fix the test on Linux.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants