Skip to content

Conversation

mahesh-attarde
Copy link
Contributor

This PR adds support for C/CPP Lib Intrinsic from LangRef in GlobalIsel.
Test is added #156281

@llvmbot
Copy link
Member

llvmbot commented Sep 1, 2025

@llvm/pr-subscribers-backend-x86

Author: Mahesh-Attarde (mahesh-attarde)

Changes

This PR adds support for C/CPP Lib Intrinsic from LangRef in GlobalIsel.
Test is added #156281


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

1 Files Affected:

  • (modified) llvm/lib/Target/X86/GISel/X86LegalizerInfo.cpp (+2-1)
diff --git a/llvm/lib/Target/X86/GISel/X86LegalizerInfo.cpp b/llvm/lib/Target/X86/GISel/X86LegalizerInfo.cpp
index 7fe58539cd4ec..c27e6075c779f 100644
--- a/llvm/lib/Target/X86/GISel/X86LegalizerInfo.cpp
+++ b/llvm/lib/Target/X86/GISel/X86LegalizerInfo.cpp
@@ -580,7 +580,8 @@ X86LegalizerInfo::X86LegalizerInfo(const X86Subtarget &STI,
       .lower();
 
   // fp intrinsics
-  getActionDefinitionsBuilder(G_INTRINSIC_ROUNDEVEN)
+  getActionDefinitionsBuilder(
+      {G_INTRINSIC_ROUNDEVEN, G_INTRINSIC_TRUNC, G_FCEIL, G_FFLOOR})
       .scalarize(0)
       .minScalar(0, LLT::scalar(32))
       .libcall();

@abhishek-kaushik22 abhishek-kaushik22 requested review from arsenm, e-kud and RKSimon and removed request for arsenm and e-kud September 6, 2025 07:08
@arsenm
Copy link
Contributor

arsenm commented Sep 6, 2025

Should be tested

@e-kud
Copy link
Contributor

e-kud commented Sep 7, 2025

@abhishek-kaushik22 @mahesh-attarde any difference from #156633?

@mahesh-attarde
Copy link
Contributor Author

@abhishek-kaushik22 @mahesh-attarde any difference from #156633?

This is not rebased to test added for same use case.
I used different sandbox to fix #156633

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