RFR: JDK-8273914: Indy string concat changes order of operations [v2]

Liam Miller-Cushon cushon at openjdk.java.net
Thu Oct 7 19:47:40 UTC 2021


> This change makes string concatenation call `toString` on the arguments eagerly, to preserve the correct evaluation order of the arguments and the calls to `toString`.

Liam Miller-Cushon has updated the pull request incrementally with two additional commits since the last revision:

 - Also test evaluation order for 'inline'
 - Remove sharpestAccessible logic, since types are now passed as strings

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/5844/files
  - new: https://git.openjdk.java.net/jdk/pull/5844/files/dd3abbe5..fabecf52

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=5844&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=5844&range=00-01

  Stats: 28 lines in 2 files changed: 4 ins; 21 del; 3 mod
  Patch: https://git.openjdk.java.net/jdk/pull/5844.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/5844/head:pull/5844

PR: https://git.openjdk.java.net/jdk/pull/5844


More information about the compiler-dev mailing list