[PATCH] D38439: AMDGPU/GlobalISel: Mark 32-bit G_FADD as legal

[Tom Stellard via Phabricator via llvm-commits]

tstellar added inline comments.


================
Comment at: test/CodeGen/AMDGPU/GlobalISel/legalize-fadd.mir:1
+# RUN: llc -mtriple=amdgcn-mesa-mesa3d -mcpu=fiji -run-pass=legalizer -global-isel %s -o - | FileCheck %s
+
----------------
kristof.beyls wrote:
> I just find "-mcpu=fiji" a bit strange for this test - I'm a bit surprised an -mcpu would be needed here.
> Otherwise, without knowing hardly anything about the AMDGPU target, the rest of this patch looks fine to me.
It's not necessary, I will drop it before I commit.


https://reviews.llvm.org/D38439