[all-commits] [llvm/llvm-project] 5aa6e2: AMDGPU/GlobalISel: Legalize f64 G_FFLOOR for SI

Matt Arsenault via All-commits all-commits at lists.llvm.org
Wed Feb 5 11:32:29 PST 2020

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 5aa6e246a1e44655a66581bc2ca6a20e3051e7e9
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2020-02-05 (Wed, 05 Feb 2020)

  Changed paths:
    M llvm/include/llvm/CodeGen/GlobalISel/MachineIRBuilder.h
    M llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
    M llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.h
    M llvm/lib/Target/AMDGPU/AMDGPUSubtarget.h
    M llvm/lib/Target/AMDGPU/SIInstructions.td
    A llvm/test/CodeGen/AMDGPU/GlobalISel/floor.f64.ll
    R llvm/test/CodeGen/AMDGPU/GlobalISel/inst-select-ffloor.mir
    A llvm/test/CodeGen/AMDGPU/GlobalISel/inst-select-ffloor.s32.mir
    A llvm/test/CodeGen/AMDGPU/GlobalISel/inst-select-ffloor.s64.mir
    M llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-ffloor.mir
    M llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-fptosi.mir
    M llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-fptoui.mir

  Log Message:
  AMDGPU/GlobalISel: Legalize f64 G_FFLOOR for SI

Use cmp ord instead of cmp_class compared to the DAG version for the
nan check, but mostly try to match the existsing pattern.

I think the sign doesn't matter for fract, so we could do a little
better with the source modifier matching.

I think this is also still broken as in D22898, but I'm leaving it
as-is for now while I don't have an SI system to test on.

More information about the All-commits mailing list