This is an archive of the discontinued LLVM Phabricator instance.

Differential D62905

[MIPS GlobalISel] Select sqrt
ClosedPublic

Authored by Petar.Avramovic on Jun 5 2019, 7:38 AM.

Details

Reviewers
atanasyan
petarj
Commits
rG81132ce0e95a: [MIPS GlobalISel] Select sqrt
rL362692: [MIPS GlobalISel] Select sqrt
Summary

Select G_FSQRT for MIPS32.

Diff Detail

Event Timeline

Petar.Avramovic created this revision.Jun 5 2019, 7:38 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 5 2019, 7:38 AM
Herald added subscribers: llvm-commits, jrtc27, arichardson and 3 others. · View Herald Transcript
atanasyan accepted this revision.Jun 5 2019, 1:13 PM

LGTM

This revision is now accepted and ready to land.Jun 5 2019, 1:13 PM
Closed by commit rL362692: [MIPS GlobalISel] Select sqrt (authored by Petar.Avramovic). · Explain WhyJun 6 2019, 2:57 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
lib/
Target/
Mips/
2 lines
3 lines
test/
CodeGen/
Mips/
GlobalISel/
instruction-select/
65 lines
legalizer/
61 lines
llvm-ir/
27 lines
regbankselect/
63 lines

Diff 203148

lib/Target/Mips/MipsLegalizerInfo.cpp

Show First 20 Lines • Show All 88 Lines • ▼ Show 20 LinesMipsLegalizerInfo::MipsLegalizerInfo(const MipsSubtarget &ST) {
getActionDefinitionsBuilder(G_GLOBAL_VALUE) getActionDefinitionsBuilder(G_GLOBAL_VALUE)
.legalFor({p0}); .legalFor({p0});
// FP instructions // FP instructions
getActionDefinitionsBuilder(G_FCONSTANT) getActionDefinitionsBuilder(G_FCONSTANT)
.legalFor({s32, s64}); .legalFor({s32, s64});
getActionDefinitionsBuilder({G_FADD, G_FSUB, G_FMUL, G_FDIV, G_FABS}) getActionDefinitionsBuilder({G_FADD, G_FSUB, G_FMUL, G_FDIV, G_FABS, G_FSQRT})
.legalFor({s32, s64}); .legalFor({s32, s64});
getActionDefinitionsBuilder(G_FCMP) getActionDefinitionsBuilder(G_FCMP)
.legalFor({{s32, s32}, {s32, s64}}) .legalFor({{s32, s32}, {s32, s64}})
.minScalar(0, s32); .minScalar(0, s32);
getActionDefinitionsBuilder({G_FCEIL, G_FFLOOR}) getActionDefinitionsBuilder({G_FCEIL, G_FFLOOR})
.libcallFor({s32, s64}); .libcallFor({s32, s64});
Show All 22 Lines

lib/Target/Mips/MipsRegisterBankInfo.cpp

Show First 20 Lines • Show All 128 Lines • ▼ Show 20 LinesMipsRegisterBankInfo::getInstrMapping(const MachineInstr &MI) const {
case G_SREM: case G_SREM:
case G_UREM: case G_UREM:
OperandsMapping = &Mips::ValueMappings[Mips::GPRIdx]; OperandsMapping = &Mips::ValueMappings[Mips::GPRIdx];
break; break;
case G_FADD: case G_FADD:
case G_FSUB: case G_FSUB:
case G_FMUL: case G_FMUL:
case G_FDIV: case G_FDIV:
case G_FABS: { case G_FABS:
case G_FSQRT:{
unsigned Size = MRI.getType(MI.getOperand(0).getReg()).getSizeInBits(); unsigned Size = MRI.getType(MI.getOperand(0).getReg()).getSizeInBits();
assert((Size == 32 || Size == 64) && "Unsupported floating point size"); assert((Size == 32 || Size == 64) && "Unsupported floating point size");
OperandsMapping = Size == 32 ? &Mips::ValueMappings[Mips::SPRIdx] OperandsMapping = Size == 32 ? &Mips::ValueMappings[Mips::SPRIdx]
: &Mips::ValueMappings[Mips::DPRIdx]; : &Mips::ValueMappings[Mips::DPRIdx];
break; break;
} }
case G_FCONSTANT: { case G_FCONSTANT: {
unsigned Size = MRI.getType(MI.getOperand(0).getReg()).getSizeInBits(); unsigned Size = MRI.getType(MI.getOperand(0).getReg()).getSizeInBits();
▲ Show 20 Lines • Show All 53 Lines • Show Last 20 Lines

test/CodeGen/Mips/GlobalISel/instruction-select/fsqrt.mir

  • This file was added.
# NOTE: Assertions have been autogenerated by utils/update_mir_test_checks.py
# RUN: llc -O0 -mtriple=mipsel-linux-gnu -run-pass=instruction-select -verify-machineinstrs %s -o - | FileCheck %s -check-prefixes=FP32
# RUN: llc -O0 -mtriple=mipsel-linux-gnu -mattr=+fp64,+mips32r2 -run-pass=instruction-select -verify-machineinstrs %s -o - | FileCheck %s -check-prefixes=FP64
--- |
define void @sqrt_f32() {entry: ret void}
define void @sqrt_f64() {entry: ret void}
...
---
name: sqrt_f32
alignment: 2
legalized: true
regBankSelected: true
tracksRegLiveness: true
body: |
bb.1.entry:
liveins: $f12
; FP32-LABEL: name: sqrt_f32
; FP32: liveins: $f12
; FP32: [[COPY:%[0-9]+]]:fgr32 = COPY $f12
; FP32: [[FSQRT_S:%[0-9]+]]:fgr32 = FSQRT_S [[COPY]]
; FP32: $f0 = COPY [[FSQRT_S]]
; FP32: RetRA implicit $f0
; FP64-LABEL: name: sqrt_f32
; FP64: liveins: $f12
; FP64: [[COPY:%[0-9]+]]:fgr32 = COPY $f12
; FP64: [[FSQRT_S:%[0-9]+]]:fgr32 = FSQRT_S [[COPY]]
; FP64: $f0 = COPY [[FSQRT_S]]
; FP64: RetRA implicit $f0
%0:fprb(s32) = COPY $f12
%1:fprb(s32) = G_FSQRT %0
$f0 = COPY %1(s32)
RetRA implicit $f0
...
---
name: sqrt_f64
alignment: 2
legalized: true
regBankSelected: true
tracksRegLiveness: true
body: |
bb.1.entry:
liveins: $d6
; FP32-LABEL: name: sqrt_f64
; FP32: liveins: $d6
; FP32: [[COPY:%[0-9]+]]:afgr64 = COPY $d6
; FP32: [[FSQRT_D32_:%[0-9]+]]:afgr64 = FSQRT_D32 [[COPY]]
; FP32: $d0 = COPY [[FSQRT_D32_]]
; FP32: RetRA implicit $d0
; FP64-LABEL: name: sqrt_f64
; FP64: liveins: $d6
; FP64: [[COPY:%[0-9]+]]:fgr64 = COPY $d6
; FP64: [[FSQRT_D64_:%[0-9]+]]:fgr64 = FSQRT_D64 [[COPY]]
; FP64: $d0 = COPY [[FSQRT_D64_]]
; FP64: RetRA implicit $d0
%0:fprb(s64) = COPY $d6
%1:fprb(s64) = G_FSQRT %0
$d0 = COPY %1(s64)
RetRA implicit $d0
...

test/CodeGen/Mips/GlobalISel/legalizer/fsqrt.mir

  • This file was added.
# NOTE: Assertions have been autogenerated by utils/update_mir_test_checks.py
# RUN: llc -O0 -mtriple=mipsel-linux-gnu -run-pass=legalizer -verify-machineinstrs %s -o - | FileCheck %s -check-prefixes=FP32
# RUN: llc -O0 -mtriple=mipsel-linux-gnu -mattr=+fp64,+mips32r2 -run-pass=legalizer -verify-machineinstrs %s -o - | FileCheck %s -check-prefixes=FP64
--- |
define void @sqrt_f32() {entry: ret void}
define void @sqrt_f64() {entry: ret void}
...
---
name: sqrt_f32
alignment: 2
tracksRegLiveness: true
body: |
bb.1.entry:
liveins: $f12
; FP32-LABEL: name: sqrt_f32
; FP32: liveins: $f12
; FP32: [[COPY:%[0-9]+]]:_(s32) = COPY $f12
; FP32: [[FSQRT:%[0-9]+]]:_(s32) = G_FSQRT [[COPY]]
; FP32: $f0 = COPY [[FSQRT]](s32)
; FP32: RetRA implicit $f0
; FP64-LABEL: name: sqrt_f32
; FP64: liveins: $f12
; FP64: [[COPY:%[0-9]+]]:_(s32) = COPY $f12
; FP64: [[FSQRT:%[0-9]+]]:_(s32) = G_FSQRT [[COPY]]
; FP64: $f0 = COPY [[FSQRT]](s32)
; FP64: RetRA implicit $f0
%0:_(s32) = COPY $f12
%1:_(s32) = G_FSQRT %0
$f0 = COPY %1(s32)
RetRA implicit $f0
...
---
name: sqrt_f64
alignment: 2
tracksRegLiveness: true
body: |
bb.1.entry:
liveins: $d6
; FP32-LABEL: name: sqrt_f64
; FP32: liveins: $d6
; FP32: [[COPY:%[0-9]+]]:_(s64) = COPY $d6
; FP32: [[FSQRT:%[0-9]+]]:_(s64) = G_FSQRT [[COPY]]
; FP32: $d0 = COPY [[FSQRT]](s64)
; FP32: RetRA implicit $d0
; FP64-LABEL: name: sqrt_f64
; FP64: liveins: $d6
; FP64: [[COPY:%[0-9]+]]:_(s64) = COPY $d6
; FP64: [[FSQRT:%[0-9]+]]:_(s64) = G_FSQRT [[COPY]]
; FP64: $d0 = COPY [[FSQRT]](s64)
; FP64: RetRA implicit $d0
%0:_(s64) = COPY $d6
%1:_(s64) = G_FSQRT %0
$d0 = COPY %1(s64)
RetRA implicit $d0
...

test/CodeGen/Mips/GlobalISel/llvm-ir/fsqrt.ll

  • This file was added.
; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
; RUN: llc -O0 -mtriple=mipsel-linux-gnu -global-isel -verify-machineinstrs %s -o -| FileCheck %s -check-prefixes=MIPS32,FP32
; RUN: llc -O0 -mtriple=mipsel-linux-gnu -mattr=+fp64,+mips32r2 -global-isel -verify-machineinstrs %s -o -| FileCheck %s -check-prefixes=MIPS32,FP64
declare float @llvm.sqrt.f32(float)
define float @sqrt_f32(float %a) {
; MIPS32-LABEL: sqrt_f32:
; MIPS32: # %bb.0: # %entry
; MIPS32-NEXT: sqrt.s $f0, $f12
; MIPS32-NEXT: jr $ra
; MIPS32-NEXT: nop
entry:
%0 = call float @llvm.sqrt.f32(float %a)
ret float %0
}
declare double @llvm.sqrt.f64(double)
define double @sqrt_f64(double %a) {
; MIPS32-LABEL: sqrt_f64:
; MIPS32: # %bb.0: # %entry
; MIPS32-NEXT: sqrt.d $f0, $f12
; MIPS32-NEXT: jr $ra
; MIPS32-NEXT: nop
entry:
%0 = call double @llvm.sqrt.f64(double %a)
ret double %0
}

test/CodeGen/Mips/GlobalISel/regbankselect/fsqrt.mir

  • This file was added.
# NOTE: Assertions have been autogenerated by utils/update_mir_test_checks.py
# RUN: llc -O0 -mtriple=mipsel-linux-gnu -run-pass=regbankselect -verify-machineinstrs %s -o - | FileCheck %s -check-prefixes=FP32
# RUN: llc -O0 -mtriple=mipsel-linux-gnu -mattr=+fp64,+mips32r2 -run-pass=regbankselect -verify-machineinstrs %s -o - | FileCheck %s -check-prefixes=FP64
--- |
define void @sqrt_f32() {entry: ret void}
define void @sqrt_f64() {entry: ret void}
...
---
name: sqrt_f32
alignment: 2
legalized: true
tracksRegLiveness: true
body: |
bb.1.entry:
liveins: $f12
; FP32-LABEL: name: sqrt_f32
; FP32: liveins: $f12
; FP32: [[COPY:%[0-9]+]]:fprb(s32) = COPY $f12
; FP32: [[FSQRT:%[0-9]+]]:fprb(s32) = G_FSQRT [[COPY]]
; FP32: $f0 = COPY [[FSQRT]](s32)
; FP32: RetRA implicit $f0
; FP64-LABEL: name: sqrt_f32
; FP64: liveins: $f12
; FP64: [[COPY:%[0-9]+]]:fprb(s32) = COPY $f12
; FP64: [[FSQRT:%[0-9]+]]:fprb(s32) = G_FSQRT [[COPY]]
; FP64: $f0 = COPY [[FSQRT]](s32)
; FP64: RetRA implicit $f0
%0:_(s32) = COPY $f12
%1:_(s32) = G_FSQRT %0
$f0 = COPY %1(s32)
RetRA implicit $f0
...
---
name: sqrt_f64
alignment: 2
legalized: true
tracksRegLiveness: true
body: |
bb.1.entry:
liveins: $d6
; FP32-LABEL: name: sqrt_f64
; FP32: liveins: $d6
; FP32: [[COPY:%[0-9]+]]:fprb(s64) = COPY $d6
; FP32: [[FSQRT:%[0-9]+]]:fprb(s64) = G_FSQRT [[COPY]]
; FP32: $d0 = COPY [[FSQRT]](s64)
; FP32: RetRA implicit $d0
; FP64-LABEL: name: sqrt_f64
; FP64: liveins: $d6
; FP64: [[COPY:%[0-9]+]]:fprb(s64) = COPY $d6
; FP64: [[FSQRT:%[0-9]+]]:fprb(s64) = G_FSQRT [[COPY]]
; FP64: $d0 = COPY [[FSQRT]](s64)
; FP64: RetRA implicit $d0
%0:_(s64) = COPY $d6
%1:_(s64) = G_FSQRT %0
$d0 = COPY %1(s64)
RetRA implicit $d0
...