[llvm] [PowerPC] Add missing patterns for lround when i32 is returned. (PR #111863)

Amy Kwan via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 10 10:32:43 PDT 2024


================
@@ -214,6 +214,48 @@ entry:
 
 declare i64 @llvm.lround.i64.f64(double)
 
+define dso_local i32 @test_lround32(double %d) local_unnamed_addr {
----------------
amy-kwan wrote:

nit: We're passing a double into the intrinsic in this test. I think the name of this test might sound a bit too similar to the other one (`test_lroundf32`). Maybe we should update this to indicate something like f64 (but returning an i32)?

Anyway, this is just a picky nit of mine but if you don't feel it is necessary you can feel free to disregard.

https://github.com/llvm/llvm-project/pull/111863


More information about the llvm-commits mailing list