Re: RFR: 8351165: Remove unused includes from vmStructs [v3]

2025-03-04 Thread Coleen Phillimore
On Tue, 4 Mar 2025 14:30:49 GMT, Kim Barrett wrote: > I think that if classes are being removed from here that the friendship with VMStructs by those classes should also be removed. Good point. Thanks for remembering these. - PR Comment: https://git.openjdk.org/jdk/pull/23897#issu

Re: RFR: 8351165: Remove unused includes from vmStructs [v3]

2025-03-04 Thread Kim Barrett
On Tue, 4 Mar 2025 13:22:24 GMT, Coleen Phillimore wrote: >> Please review this trivial change to remove some unneeded includes and type >> declarations from vmStructs. The SystemDictionary type is unused because >> classes are looked up in the ClassLoaderDataGraph in the SA, and they >> have

Re: RFR: 8351165: Remove unused includes from vmStructs [v3]

2025-03-04 Thread Coleen Phillimore
> Please review this trivial change to remove some unneeded includes and type > declarations from vmStructs. The SystemDictionary type is unused because > classes are looked up in the ClassLoaderDataGraph in the SA, and they haven't > been in a single SystemDictionary for a very long time. > Te