On Thu, 7 May 2026 06:04:14 GMT, Axel Boldt-Christmas <[email protected]> wrote:
> VM_PrintClassLayout uses the wrong VMOp_Type. > > Probably a copy paste error. Uses `VMOp_PrintClassHierarchy` instead of > `VMOp_ClassPrintLayout`. > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). src/hotspot/share/runtime/vmOperations.hpp line 283: > 281: public: > 282: VM_PrintClassLayout(outputStream* st, char* class_name): _out(st), > _class_name(class_name) {} > 283: VMOp_Type type() const { return VMOp_ClassPrintLayout; } Maybe pre-existing, but it looks inconsistent to me that the the op is named `ClassPrintLayout` instead of `PrintClassLayout`. Other operations in this file mirror the class name in the op name. ------------- PR Review Comment: https://git.openjdk.org/valhalla/pull/2401#discussion_r3199425700
