[AArch64][v8.5A] Add FRINT[32,64][Z,X] instructions
ClosedPublic
Actions

Authored by olista01 on Sep 25 2018, 7:16 AM.

Details

Reviewers

javed.absar
t.p.northover
rengolin
SjoerdMeijer
ab

Commits

rGddb7d46aa53c: [AArch64][v8.5A] Add FRINT[32,64][Z,X] instructions
rL343209: [AArch64][v8.5A] Add FRINT[32,64][Z,X] instructions

Summary

These are some new variants of the "Floating-point Round to Integral"
family of instructions, which round to the nearest floating-point value
which fits in a 32- or 64-bit integer.

Patch by Pablo Barrio!

Diff Detail

Repository: rL LLVM

Event Timeline

olista01 created this revision.Sep 25 2018, 7:16 AM

Herald added a reviewer: javed.absar. · View Herald TranscriptSep 25 2018, 7:16 AM

Herald added subscribers: llvm-commits, kristof.beyls. · View Herald Transcript

Harbormaster completed remote builds in B23068: Diff 166892.Sep 25 2018, 7:17 AM

olista01 added reviewers: t.p.northover, rengolin, SjoerdMeijer, ab.Sep 25 2018, 7:23 AM

t.p.northover added inline comments.Sep 26 2018, 8:01 AM

lib/Target/AArch64/AArch64InstrInfo.td
67 ↗	(On Diff #166892)	What's behind the "NNT" here? It's not a particularly obvious name.

olista01 added inline comments.Sep 27 2018, 2:24 AM

lib/Target/AArch64/AArch64InstrInfo.td
67 ↗	(On Diff #166892)	I've asked Pablo (who originally wrote this patch) and he can't remember what it stands for, so I agree we should change it so something more meaningful. How about HasFRInt3264?