RFR: 8282365: Consolidate and improve division by constant idealizations [v42]
Stefan Karlsson
stefank at openjdk.org
Tue Jan 9 16:00:01 UTC 2024
On Tue, 9 Jan 2024 15:29:44 GMT, Quan Anh Mai <qamai at openjdk.org> wrote:
>> In the Oracle-internal discussion of include order from about a year ago, there was not a consensus
>> decision about the position of "unittest.hpp". There was a concern that in some cases it really was
>> required to be last for some technical reason. That needed (and still needs) investigation.
>
> I assume this means that the include order is good as it is now?
Please update it to:
#include "precompiled.hpp"
#include "opto/divconstants.hpp"
#include "runtime/os.hpp"
#include "utilities/growableArray.hpp"
#include "unittest.hpp"
#include <type_traits>
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/9947#discussion_r1446279651
More information about the hotspot-compiler-dev
mailing list