Depends On D96040
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
mlir/lib/Dialect/Vector/VectorOps.cpp | ||
---|---|---|
2826 | I believe passing the value as float can be risky as Nan encoding can be changed. I'm not positive this is a problem here but it might be better to create an APFloat directly from bits? |
Comment Actions
Address comments
mlir/lib/Dialect/Vector/VectorOps.cpp | ||
---|---|---|
2826 | Good point! Changed to directly use APFloat. |
I believe passing the value as float can be risky as Nan encoding can be changed. I'm not positive this is a problem here but it might be better to create an APFloat directly from bits?