Skip to content

Conversation

Prabhuk
Copy link
Contributor

@Prabhuk Prabhuk commented Aug 28, 2025

Simplify call-graph-section.ll test file for better readability.

Simplify  call-graph-section.ll test file for better readability.
@llvmbot
Copy link
Member

llvmbot commented Aug 28, 2025

@llvm/pr-subscribers-backend-x86

Author: Prabhu Rajasekaran (Prabhuk)

Changes

Simplify call-graph-section.ll test file for better readability.


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

1 Files Affected:

  • (modified) llvm/test/CodeGen/X86/call-graph-section.ll (+2-4)
diff --git a/llvm/test/CodeGen/X86/call-graph-section.ll b/llvm/test/CodeGen/X86/call-graph-section.ll
index 4a9840eac4898..66d009cf1221d 100644
--- a/llvm/test/CodeGen/X86/call-graph-section.ll
+++ b/llvm/test/CodeGen/X86/call-graph-section.ll
@@ -11,14 +11,12 @@ declare !type !2 ptr @baz(ptr)
 
 define void @main() {
 entry:
-  %a = alloca i8, align 1
   %fp_foo_val = load ptr, ptr null, align 8
   call void (...) %fp_foo_val(), !callee_type !1
   %fp_bar_val = load ptr, ptr null, align 8
-  %param = trunc i64 0 to i8
-  %call_fp_bar = call i32 %fp_bar_val(i8 signext %param), !callee_type !3
+  %call_fp_bar = call i32 %fp_bar_val(i8 0), !callee_type !3
   %fp_baz_val = load ptr, ptr null, align 8
-  %call_fp_baz = call ptr %fp_baz_val(ptr %a), !callee_type !4
+  %call_fp_baz = call ptr %fp_baz_val(ptr null), !callee_type !4
   ret void
 }
 

@Prabhuk Prabhuk requested a review from ilovepi August 28, 2025 21:32
Copy link
Contributor

@ilovepi ilovepi left a comment

Choose a reason for hiding this comment

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

LGTM. CI failures seem unrelated, so a rebase will probably fix.

@Prabhuk Prabhuk merged commit 6980d3b into llvm:main Sep 1, 2025
9 checks passed
@Prabhuk Prabhuk deleted the Prabhuk-patch-1 branch October 21, 2025 16:57
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.

3 participants