[PATCH] D25999: LegalizeDAG: Implement expansion of f16 = FP_TO_FP16 f64

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 31 14:10:53 PDT 2016


arsenm added a comment.

LGTM with the test fixed



================
Comment at: test/CodeGen/AMDGPU/trunc-store-f64-to-f16.ll:1
-; XFAIL: *
 ; RUN: llc -march=amdgcn -mcpu=SI < %s
 
----------------
Can you remove the -mcpu and add -verify-machineinstrs, also the FileCheck part is missing


https://reviews.llvm.org/D25999





More information about the llvm-commits mailing list