On Wed, 1 Mar 2023 15:42:08 GMT, Raffaello Giulietti <[email protected]>
wrote:
>> Joe Darcy has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Add note explaining goto translation.
>
> test/jdk/java/lang/StrictMath/FdlibmTranslit.java line 867:
>
>> 865:
>> 866: jz = jk;
>> 867: while(true) {
>
> Would it be worthwhile to add some remarks about how the C `goto` + label has
> been "translitted"?
> Or even add the `recompute:` label explicitly and adapt the `continue` and
> `break` statements inside the `while`?
Note added. I think there is a stronger case for adding the note to the
transliteration port than to the mainline one.
I'm open to restructuring the looping construct for the mainline port.
-------------
PR: https://git.openjdk.org/jdk/pull/12800