On Thu, 15 Feb 2024 14:13:19 GMT, Magnus Ihse Bursie <i...@openjdk.org> wrote:
> This is a follow-up to > [JDK-8325877](https://bugs.openjdk.org/browse/JDK-8325877). I was careful in > that bug not to change order of any lines, only split up the original file > into parts. > > In this PR, I use the new structure to improve the design. I collect the > functionality into three clearly separate phases, preparation, compilation > and linking. I use consistent naming to reveal whether a function just sets > up variables, or create output. I simplify and split up a few extra hard to > read functions. I move some code around so it is placed more logically. > > It can be hard to convince yourself that the changes are correct if you just > look at the end result. I have tried to make small and clear changes in each > separate commit, and to explain in the commit title what I am doing. I > recommend reviewing this PR [commit by > commit](https://github.com/openjdk/jdk/pull/17873/commits). This pull request has now been integrated. Changeset: 1bd91cde Author: Magnus Ihse Bursie <i...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/1bd91cdebee1e9ec78ecf185529923eef40ff89c Stats: 389 lines in 6 files changed: 134 ins; 130 del; 125 mod 8325963: Clean up NativeCompilation.gmk and its newly created parts Reviewed-by: jwaters, erikj ------------- PR: https://git.openjdk.org/jdk/pull/17873
