RFR: JDK-8301498: Replace NULL with nullptr in cpu/x86 [v2]

Vladimir Kozlov kvn at openjdk.org
Tue Feb 14 19:19:49 UTC 2023


On Mon, 6 Feb 2023 10:21:25 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

>> Johan Sjölen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Some more fixes
>
> src/hotspot/cpu/x86/interp_masm_x86.cpp line 402:
> 
>> 400:     movptr(tmp, Address(rthread, JavaThread::jvmti_thread_state_offset()));
>> 401:     testptr(tmp, tmp);
>> 402:     jcc(Assembler::zero, L); // if (thread->jvmti_thread_state() == null) exit;
> 
> nullptr

Missed change to nullptr as you suggested.

> src/hotspot/cpu/x86/interp_masm_x86.cpp line 1779:
> 
>> 1777: //   // main copy of decision tree, rooted at row[1]
>> 1778: //   if (row[0].rec == rec) { row[0].incr(); goto done; }
>> 1779: //   if (row[0].rec != null) {
> 
> nullptr

Missed change to nullptr as you suggested.

> src/hotspot/cpu/x86/macroAssembler_x86.cpp line 4265:
> 
>> 4263:   }
>> 4264: 
>> 4265:   // for (scan = klass->itable(); scan->interface() != null; scan += scan_step) {
> 
> nullptr

Missed change to nullptr as you suggested.

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

PR: https://git.openjdk.org/jdk/pull/12326


More information about the shenandoah-dev mailing list