[PATCH] D130151: [X86][FP16] Do not split FP64->FP16 to FP64->FP32->FP16

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 21 07:03:32 PDT 2022


RKSimon added inline comments.


================
Comment at: llvm/test/CodeGen/X86/cvt16.ll:24
 
-define void @test1(float %src, ptr %dest) {
+define void @test1(float %src, ptr %dest) nounwind {
 ; LIBCALL-LABEL: test1:
----------------
apply the nounwind change separately to minimise diff


================
Comment at: llvm/test/CodeGen/X86/fastmath-float-half-conversion.ll:93
-attributes #0 = { nounwind readnone uwtable "unsafe-fp-math"="true" "use-soft-float"="false" }
-attributes #1 = { nounwind readnone uwtable "unsafe-fp-math"="false" "use-soft-float"="false" }
----------------
apply separately?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130151/new/

https://reviews.llvm.org/D130151



More information about the llvm-commits mailing list