This is an archive of the discontinued LLVM Phabricator instance.

Differential D28702

AMDGPU: Remove dead pattern
ClosedPublic

Authored by arsenm on Jan 13 2017, 2:10 PM.

Details

Reviewers
• tstellarAMD
kzhuravl
Summary

This is the unsafe conversion pattern, but not guarded by
an unsafe math check. It is also already done in LegalizeDAG.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 84378.Jan 13 2017, 2:10 PM
arsenm retitled this revision from to AMDGPU: Remove dead pattern.
arsenm updated this object.
arsenm added a subscriber: llvm-commits.
Herald added a reviewer: • tstellarAMD. · View Herald TranscriptJan 13 2017, 2:10 PM
Herald added subscribers: tony-tye, yaxunl, nhaehnle and 2 others. · View Herald Transcript
kzhuravl accepted this revision.Jan 13 2017, 2:36 PM
kzhuravl added a reviewer: kzhuravl.

LGTM

This revision is now accepted and ready to land.Jan 13 2017, 2:36 PM
arsenm closed this revision.Jan 16 2017, 4:22 PM

r292173

Revision Contents

PathSize
lib/
Target/
AMDGPU/
5 lines

Diff 84378

lib/Target/AMDGPU/SIInstructions.td

Show First 20 Lines • Show All 451 Lines • ▼ Show 20 Lines
// fp_to_fp16 patterns // fp_to_fp16 patterns
def : Pat < def : Pat <
(i32 (fp_to_f16 (f32 (VOP3Mods0 f32:$src0, i32:$src0_modifiers, i1:$clamp, i32:$omod)))), (i32 (fp_to_f16 (f32 (VOP3Mods0 f32:$src0, i32:$src0_modifiers, i1:$clamp, i32:$omod)))),
(V_CVT_F16_F32_e64 $src0_modifiers, f32:$src0, $clamp, $omod) (V_CVT_F16_F32_e64 $src0_modifiers, f32:$src0, $clamp, $omod)
>; >;
def : Pat < def : Pat <
(f16 (fpround f64:$src)),
(V_CVT_F16_F32_e32 (V_CVT_F32_F64_e32 $src))
>;
def : Pat <
(i32 (fp_to_sint f16:$src)), (i32 (fp_to_sint f16:$src)),
(V_CVT_I32_F32_e32 (V_CVT_F32_F16_e32 $src)) (V_CVT_I32_F32_e32 (V_CVT_F32_F16_e32 $src))
>; >;
def : Pat < def : Pat <
(i32 (fp_to_uint f16:$src)), (i32 (fp_to_uint f16:$src)),
(V_CVT_U32_F32_e32 (V_CVT_F32_F16_e32 $src)) (V_CVT_U32_F32_e32 (V_CVT_F32_F16_e32 $src))
>; >;
▲ Show 20 Lines • Show All 640 Lines • Show Last 20 Lines