This patch fixes the floating point conversion warnings found with
-Wconversion and -Wno-sign-conversion. These were the last warnings
I found, meaning that once this lands https://reviews.llvm.org/D156630
should be unblocked.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
The literals here use the f suffix to have float type, but then the variable is double.
Should the variable just be float? And I wonder too why it's not constexpr.