Re: RFR: 8348180: Remove mention of include of precompiled.hpp from the HotSpot Style Guide [v2]

2025-01-29 Thread Stefan Karlsson
On Wed, 22 Jan 2025 09:34:18 GMT, Stefan Karlsson wrote: >> [JDK-8347909](https://bugs.openjdk.org/browse/JDK-8347909) removed the need >> for HotSpot devs to include the precompiled.hpp. This is the PR to reflect >> that. >> >> I know that there is some process around updating the style guide

Re: RFR: 8348180: Remove mention of include of precompiled.hpp from the HotSpot Style Guide [v2]

2025-01-22 Thread David Holmes
On Wed, 22 Jan 2025 09:34:18 GMT, Stefan Karlsson wrote: >> [JDK-8347909](https://bugs.openjdk.org/browse/JDK-8347909) removed the need >> for HotSpot devs to include the precompiled.hpp. This is the PR to reflect >> that. >> >> I know that there is some process around updating the style guide

Re: RFR: 8348180: Remove mention of include of precompiled.hpp from the HotSpot Style Guide [v2]

2025-01-22 Thread Stefan Karlsson
> [JDK-8347909](https://bugs.openjdk.org/browse/JDK-8347909) removed the need > for HotSpot devs to include the precompiled.hpp. This is the PR to reflect > that. > > I know that there is some process around updating the style guide, but I > can't find it, so I'm starting with a plain PR. Stef

Re: RFR: 8348180: Remove mention of include of precompiled.hpp from the HotSpot Style Guide

2025-01-22 Thread Stefan Karlsson
On Tue, 21 Jan 2025 12:29:33 GMT, Stefan Karlsson wrote: > [JDK-8347909](https://bugs.openjdk.org/browse/JDK-8347909) removed the need > for HotSpot devs to include the precompiled.hpp. This is the PR to reflect > that. > > I know that there is some process around updating the style guide, but

Re: RFR: 8348180: Remove mention of include of precompiled.hpp from the HotSpot Style Guide

2025-01-21 Thread Aleksey Shipilev
On Tue, 21 Jan 2025 15:54:06 GMT, Stefan Karlsson wrote: > Maybe something like this: > > * Some build configurations uses precompiled headers to speed up the > build times. The compiled headers are included in the precompiled.hpp file. > Note that precompiled.hpp is just a build time opti

Re: RFR: 8348180: Remove mention of include of precompiled.hpp from the HotSpot Style Guide

2025-01-21 Thread Stefan Karlsson
On Tue, 21 Jan 2025 15:08:39 GMT, Aleksey Shipilev wrote: > This part looks dangling now: > > ``` > * precompiled.hpp is just a build time optimization, so don't rely on > it to resolve include problems. > ``` > > Maybe move the line at the end of the list and phrase it as: > > "* Consider ad

Re: RFR: 8348180: Remove mention of include of precompiled.hpp from the HotSpot Style Guide

2025-01-21 Thread Aleksey Shipilev
On Tue, 21 Jan 2025 12:29:33 GMT, Stefan Karlsson wrote: > [JDK-8347909](https://bugs.openjdk.org/browse/JDK-8347909) removed the need > for HotSpot devs to include the precompiled.hpp. This is the PR to reflect > that. > > I know that there is some process around updating the style guide, but

Re: RFR: 8348180: Remove mention of include of precompiled.hpp from the HotSpot Style Guide

2025-01-21 Thread Erik Ă–sterlund
On Tue, 21 Jan 2025 12:29:33 GMT, Stefan Karlsson wrote: > [JDK-8347909](https://bugs.openjdk.org/browse/JDK-8347909) removed the need > for HotSpot devs to include the precompiled.hpp. This is the PR to reflect > that. > > I know that there is some process around updating the style guide, but

RFR: 8348180: Remove mention of include of precompiled.hpp from the HotSpot Style Guide

2025-01-21 Thread Stefan Karlsson
[JDK-8347909](https://bugs.openjdk.org/browse/JDK-8347909) removed the need for HotSpot devs to include the precompiled.hpp. This is the PR to reflect that. I know that there is some process around updating the style guide, but I can't find it, so I'm starting with a plain PR. - Co