[lworld] RFR: 8351979: Improve verification error messages for strict fields

Coleen Phillimore coleenp at openjdk.org
Fri Mar 14 16:17:13 UTC 2025


On Fri, 14 Mar 2025 16:00:50 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:

> This patch adds more detail to the verifier errors throws when there are problems with strict instance fields. The class name and one field name is now printed. Verified with tier 1-3 tests.

src/hotspot/share/classfile/stackMapFrame.cpp line 71:

> 69:     sig->as_C_string(),
> 70:     klass->name()->as_C_string()
> 71:   );

I wonder if you need a ResourceMark here.  Also, I wonder if you could use a stringStream here and concatenate all the names and signatures of the unset fields?

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1400#discussion_r1995860689


More information about the valhalla-dev mailing list