> When `left == right` is known at code generation time, the operands are 
> necessarily the same oop, so we can jump directly to `L_oops_equal`. 
> This patch changes both x86 and aarch65 to do that.
> 
> I also considered emitting `equal_result` and returning early in that case.
> I decided not to do that because `LIR_OpSubstitutabilityCheck::emit_code()` 
> still appends the slow-path stub after `emit_opSubstitutabilityCheck()` 
> returns.
> An early return here would skip binding `op->stub()->continuation()`, leaving 
> the appended stub with an unbound continuation label. 
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Daisuke Yamazaki has updated the pull request incrementally with one additional 
commit since the last revision:

  Clean up the platform-specific codes

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/2628/files
  - new: https://git.openjdk.org/valhalla/pull/2628/files/3d82eb0b..91012afd

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2628&range=03
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2628&range=02-03

  Stats: 25 lines in 3 files changed: 0 ins; 16 del; 9 mod
  Patch: https://git.openjdk.org/valhalla/pull/2628.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2628/head:pull/2628

PR: https://git.openjdk.org/valhalla/pull/2628

Reply via email to