Add builtin f80 and f128 following @schweitz proposition
https://llvm.discourse.group/t/rfc-adding-better-support-for-higher-precision-floating-point/2526/5
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Did a pass through, and looks good to me.
mlir/test/Dialect/LLVMIR/types.mlir | ||
---|---|---|
15–18 | Can you add a usage in IR/parser.mlir now that these are builtin types? All of these checks here look like they should go away now that the types have been replaced. |
Add usage in IR/parser.mlir
mlir/test/Dialect/LLVMIR/types.mlir | ||
---|---|---|
15–18 | Should we also remove the checks for f16, bf16, f32, f64? |
mlir/test/Dialect/LLVMIR/types.mlir | ||
---|---|---|
15–18 | Yeah, I don't think any of those are useful anymore. |
mlir/test/Dialect/LLVMIR/types.mlir | ||
---|---|---|
15–18 | Ok. Thanks for the quick review. I just removed the checks. |
mlir/docs/Dialects/LLVM.md | ||
---|---|---|
217 | Please add f80 and f128 here |
Please add f80 and f128 here