RFR: 8326135: Enhance adlc to report unused operands [v4]
Vladimir Kozlov
kvn at openjdk.org
Thu Feb 22 07:41:56 UTC 2024
On Thu, 22 Feb 2024 06:27:12 GMT, kuaiwei <duke at openjdk.org> wrote:
>> src/hotspot/share/adlc/archDesc.cpp line 27:
>>
>>> 25:
>>> 26: // archDesc.cpp - Internal format for architecture definition
>>> 27: #include <unordered_set>
>>
>> Even though it's not hotspot code per se (but a tool used during build), it'll be the first usage of C++ STL in the hotspot code base.
>>
>> HotSpot Coding Style [1] clearly states "Avoid using the C++ Standard Library."
>>
>> @kimbarrett what are your thoughts on it?
>>
>> [1] https://github.com/openjdk/jdk/blob/master/doc/hotspot-style.md
>
> I think adlc is almost a stand alone tool, it only depends on src/hotspot/share/opto/opcodes.cpp . Could we move it to src/utils?
Why you want to move it? It depends on C2 sources and used only for VM build.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17910#discussion_r1498785906
More information about the hotspot-compiler-dev
mailing list