RFR (S): 8015266: fix some -Wsign-compare warnings in adlc
Vladimir Kozlov
vladimir.kozlov at oracle.com
Wed May 29 11:27:08 PDT 2013
On 5/23/13 11:28 AM, Christian Thalinger wrote:
> http://javaweb.us.oracle.com/~cthaling/webrev/8015266/webrev/
>
> 8015266: fix some -Wsign-compare warnings in adlc
> Reviewed-by:
>
> Some warnings I've found while hacking on the build system.
>
In general it is good. could you fix code style (spaces in if() and
for()) around codes you fixed?
> src/share/vm/adlc/archDesc.cpp
> src/share/vm/adlc/dict2.cpp
Align comment:
// Characters are always odd!
> src/share/vm/adlc/formssel.cpp
> src/share/vm/adlc/formssel.hpp
> src/share/vm/adlc/output_c.cpp
>
thanks,
Vladimir
More information about the hotspot-compiler-dev
mailing list