Page MenuHomePhabricator
Differential D143008

[mlir] Add support for f8 data types to LLVM dialect types
ClosedPublic
Actions

Authored by ezhulenev on Jan 31 2023, 12:16 PM.

Details

Reviewers
ftynse
nicolasvasilache
dcaballe
reedwm
mpaszkowski
Commits
rGfaf697e49bad: [mlir] Add support for f8 data types to LLVM dialect types
Summary

This change allows using fp8 pointers when exporting to LLVM, because we anyway export them as opaque pointers, however full support of fp8 types is not yet implemented on the LLVM side.

Diff Detail

Event Timeline

ezhulenev created this revision.Jan 31 2023, 12:16 PM
Herald added a reviewer: ftynse. · View Herald TranscriptJan 31 2023, 12:16 PM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added subscribers: Moerafaat, zero9178, bzcheeseman and 22 others. · View Herald Transcript
ezhulenev requested review of this revision.Jan 31 2023, 12:16 PM
Herald added a reviewer: nicolasvasilache. · View Herald TranscriptJan 31 2023, 12:16 PM
Herald added a reviewer: dcaballe. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
Herald added subscribers: stephenneuendorffer, nicolasvasilache. · View Herald Transcript
ezhulenev updated this revision to Diff 493707.Jan 31 2023, 12:26 PM

Do not add f8 to supported floating point types

ezhulenev edited the summary of this revision. (Show Details)Jan 31 2023, 12:29 PM
Herald added a reviewer: mpaszkowski. · View Herald TranscriptJan 31 2023, 12:29 PM
ezhulenev edited reviewers, added: reedwm; removed: mpaszkowski.Jan 31 2023, 12:30 PM
Herald added a reviewer: mpaszkowski. · View Herald TranscriptJan 31 2023, 12:30 PM
Harbormaster completed remote builds in B211056: Diff 493707.Jan 31 2023, 1:08 PM
ftynse added a comment.Feb 1 2023, 2:40 AM

Could you add a test?

ezhulenev updated this revision to Diff 494064.Feb 1 2023, 1:49 PM

Add a test for lowering f8 pointers to LLVM IR

Harbormaster completed remote builds in B211308: Diff 494064.Feb 1 2023, 3:06 PM
ezhulenev updated this revision to Diff 498359.Feb 17 2023, 7:12 AM

Rebased

ezhulenev updated this revision to Diff 498360.Feb 17 2023, 7:14 AM

Remove accidental change

Harbormaster completed remote builds in B214406: Diff 498360.Feb 17 2023, 7:43 AM
ezhulenev updated this revision to Diff 503789.Mar 9 2023, 8:28 AM

Rebased

Harbormaster completed remote builds in B218409: Diff 503789.Mar 9 2023, 9:15 AM
ezhulenev updated this revision to Diff 503873.Mar 9 2023, 11:59 AM

Rebased

Harbormaster completed remote builds in B218469: Diff 503873.Mar 9 2023, 1:50 PM
ezhulenev updated this revision to Diff 503994.Mar 9 2023, 5:47 PM

Rebased

Harbormaster completed remote builds in B218568: Diff 503994.Mar 9 2023, 6:26 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 9 2023, 6:52 PM
Closed by commit rGfaf697e49bad: [mlir] Add support for f8 data types to LLVM dialect types (authored by ezhulenev). · Explain Why
This revision was automatically updated to reflect the committed changes.
ezhulenev added a commit: rGfaf697e49bad: [mlir] Add support for f8 data types to LLVM dialect types.

Revision Contents
Changeset List

PathSize
mlir/
lib/
Dialect/
LLVMIR/
IR/
4 lines
test/
Target/
LLVMIR/
7 lines

Diff 498359

View Options

mlir/lib/Dialect/LLVMIR/IR/LLVMTypes.cpp

Loading...
View Options

mlir/test/Target/LLVMIR/llvmir.mlir

Loading...