Skip to content

Conversation

@google-yfyang
Copy link
Contributor

@llvmbot
Copy link
Member

llvmbot commented Oct 31, 2025

@llvm/pr-subscribers-lldb

Author: None (google-yfyang)

Changes

Fix the breakage in: https://buildkite.com/llvm-project/upstream-bazel/builds/155695/steps/canvas?sid=019a39c6-6ccd-4845-bd5e-d36e4f0369e2


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

1 Files Affected:

  • (modified) lldb/source/Plugins/ABI/ARC/ABISysV_arc.cpp (+1-3)
diff --git a/lldb/source/Plugins/ABI/ARC/ABISysV_arc.cpp b/lldb/source/Plugins/ABI/ARC/ABISysV_arc.cpp
index f9c249d7fec1c..4fdff0851d715 100644
--- a/lldb/source/Plugins/ABI/ARC/ABISysV_arc.cpp
+++ b/lldb/source/Plugins/ABI/ARC/ABISysV_arc.cpp
@@ -480,11 +480,9 @@ ABISysV_arc::GetReturnValueObjectSimple(Thread &thread,
   }
   // Floating point return type.
   else if (type_flags & eTypeIsFloat) {
-    uint32_t float_count = 0;
     bool is_complex = false;
 
-    if (compiler_type.IsFloatingPointType(float_count, is_complex) &&
-        1 == float_count && !is_complex) {
+    if (compiler_type.IsFloatingPointType(is_complex) && !is_complex) {
       const size_t byte_size =
           llvm::expectedToOptional(compiler_type.GetByteSize(&thread))
               .value_or(0);

Copy link
Member

@Michael137 Michael137 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! LGTM modulo comment

@rupprecht rupprecht enabled auto-merge (squash) October 31, 2025 14:20
@rupprecht rupprecht merged commit d1fa435 into llvm:main Oct 31, 2025
7 of 9 checks passed
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.

4 participants