RFR: 8150426: Wrong cast in metadata_at_put

David Holmes david.holmes at oracle.com
Tue Feb 23 02:36:28 UTC 2016


Hi Kim,

On 23/02/2016 11:57 AM, Kim Barrett wrote:
> Please review this fix of an incorrect cast.
>
> This fix was contributed by Timo Kinnunen.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8150426
>
> Webrev:
> http://cr.openjdk.java.net/~kbarrett/8150426/webrev/

Fix seems fine, but have to wonder given:

  128 #ifdef _LP64
  129   Metadata* metadata_at(int which) const {
  130     return (Metadata*)*long_at_addr(which); }
  131   void metadata_at_put(int which, Metadata* contents) {
  132     *long_at_addr(which) = (jlong)contents;
  133   }
  134 #else

is Windows setting _LP64 even though it is LLP64? Or are we 
(incorrectly) forcing _LP64 for all 64-bit builds?

Thanks,
David



More information about the hotspot-runtime-dev mailing list