RFR: 8326983: Unused operands reported after JDK-8326135 [v2]

Vladimir Kozlov kvn at openjdk.org
Mon Mar 4 18:20:53 UTC 2024


On Mon, 4 Mar 2024 10:55:22 GMT, kuaiwei <duke at openjdk.org> wrote:

> I'm not clear about m4 file. How do we use it?
> The build script of jdk will combine all ad file into one single file and adlc will compile it. So aarch64_vector.ad will be checked as well.

Aarch64 m4 files are used to manually update .ad files. My concern was that could be overlapped code in m4 files which may overwrite your changes in .ad when someone do such manual update in a future. 
Fortunately `aarch64_ad.m4` does not have operand definitions so your changes are fine.
But `aarch64_vector_ad.m4` has them so if  we need to change `aarch64_vector.ad` we need to modify m4 file too.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/18075#issuecomment-1977189798


More information about the hotspot-compiler-dev mailing list