RFR: 8282365: Consolidate and improve division by constant idealizations [v35]
Quan Anh Mai
qamai at openjdk.org
Sun Dec 24 00:07:17 UTC 2023
On Mon, 18 Dec 2023 17:07:32 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
>> I see there are multiple cases where a header is defined in multiple source files, and these are used exclusively for `DivNode`s so putting them here seems logical.
>
> If it were true these are only used in divnode.cpp then the declarations could
> be in that file. But these are also used in the gtest. That gtest duplicates
> the declaration. Better would be for it to include the suggested
> divconstants.hpp.
>
> Having examples of an unusual style doesn't mean we want more, at least not
> without some fairly good reason.
I see, I moved it to another file then.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/9947#discussion_r1435742851
More information about the hotspot-compiler-dev
mailing list