[PATCH] D28496: [AMDGPU] Implement f16 fcopysign

[Konstantin Zhuravlyov via Phabricator via llvm-commits]

kzhuravl added inline comments.


================
Comment at: test/CodeGen/AMDGPU/fcopysign.f16.ll:1
+; RUN: llc -march=amdgcn -mcpu=tonga -enable-unsafe-fp-math -verify-machineinstrs < %s | FileCheck -check-prefix=FUNC -check-prefix=GCN -check-prefix=VI %s
+
----------------
arsenm wrote:
> This should also have an SI run line. The -enable-unsafe-fp-math should also be dropped
Should check lines also be added for SI? Or run line is sufficient?


https://reviews.llvm.org/D28496