RFR: 8282365: Consolidate and improve division by constant idealizations [v33]
Quan Anh Mai
qamai at openjdk.org
Thu Jan 25 11:59:52 UTC 2024
On Mon, 30 Oct 2023 15:38:52 GMT, Raffaello Giulietti <rgiulietti at openjdk.org> wrote:
>> Quan Anh Mai has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 74 commits:
>>
>> - fix proof
>> - Merge branch 'master' into unsignedDiv
>> - fix assert macro, benchmarks
>> - comment styles
>> - disable test with Xcomp
>> - remove verify
>> - fix x86 test
>> - more rigorous control
>> - verify the effectiveness of test
>> - require x64
>> - ... and 64 more: https://git.openjdk.org/jdk/compare/5224e979...529bd0f9
>
> I agree this is not necessarily optimal, but it's far easier to prove and to code.
>
> There's a tradeoff here between simplicity of the proofs and of the code on one side, and squeezing out the last sub-nanosecond on the other side.
> Hard choice!
@rgiulietti Thanks very much for your reviews
@vnkozlov @eme64 Could you do another round of reviews, please? There has not been much change, though.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/9947#issuecomment-1910035550
More information about the hotspot-compiler-dev
mailing list