RFR (XS): 8043207: Add const to Address argument for Assembler::swap
Mikael Vidstedt
mikael.vidstedt at oracle.com
Thu May 29 20:12:03 UTC 2014
Since this is a build problem the testing pretty much comes with
building on the relevant platforms, which all pass with the fix in place.
Cheers,
Mikael
On 2014-05-29 13:04, David Chase wrote:
> Any testing? (and except for that issue, I, a non-Reviewer, approve)
>
> On 2014-05-28, at 12:17 PM, Mikael Vidstedt <mikael.vidstedt at oracle.com> wrote:
>
>> Please review the following fix which changes one of the Assembler::swap methods to take a *const* Address argument instead of a non-const. Some of the usages of the swap method pass a const argument, and when compiling with gcc the const-ness mismatch means the method cannot be resolved which in turn means a compilation error. This is for jdk7u (only) - the relevant code has been rewritten in jdk8+ to no longer have this problem.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8043207
>> Webrev: http://cr.openjdk.java.net/~mikael/webrevs/8043207/webrev.00/webrev/
>>
>> Thanks,
>> Mikael
>>
More information about the hotspot-compiler-dev
mailing list