> Making the conversion explicit would fix it.
> 
> Flag '-Wimplicit-int-float-conversion' is first introduced in clang-10.
> Therefore clang-8 and clang-9 are not affected. The flag with similar
> functionality in gcc is '-Wfloat-conversion', but it is not enabled by
> '-Wall' or '-Wextra'. That's why this warning does not appear when
> building with gcc.
> 
> 
> Note that we have tested with this patch, debug build succeeded with clang-10 
> on Linux X86-64/AArch64 machines.
> Note that '--with-extra-cxxflags=-Wno-deprecated-copy' should be added when 
> configuration. It's another issue (See JDK-8258010)

Hao Sun has updated the pull request incrementally with one additional commit 
since the last revision:

  Use static_cast<double>
  
  To match the style in this file, use static_cast to perform type
  casting.

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/1956/files
  - new: https://git.openjdk.java.net/jdk/pull/1956/files/99259621..5a30354c

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1956&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1956&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/1956.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/1956/head:pull/1956

PR: https://git.openjdk.java.net/jdk/pull/1956

Reply via email to