[lworld] RFR: 8263900: [lworld] Make .default a separate node type in the parser.

Srikanth Adayapalam sadayapalam at openjdk.java.net
Thu Mar 25 06:20:00 UTC 2021


On Thu, 25 Mar 2021 06:05:27 GMT, Srikanth Adayapalam <sadayapalam at openjdk.org> wrote:

> > Regarding (7), I hit that earlier as well - but it is a entirely seperate bug, also available in the released product:
> 
> [...]
> 
> > While I might be able to fix this locally, I think it should really be addressed in the main repo first. I tried finding the bug for it, but didn't find it -- would you like me to report it?
> 
> I agree this should be addressed in mainline first, yes please, could you raise a defect report for that ? Thanks!

Actually, I think with your work to split Foo.default away from JCFieldAccess, this should be fixed independently in both mainline (for java.lang.class example you report) and in Valhalla for java.lang.default case we are discussing. With the code split, fixing mainline won't automatically fix Valhalla issue with java.lang.default. 

However, I am fine with that being handled on a separate ticket by itself without blocking or side tracking this PR/RFR and also the work on treating .default as a poly expression.

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

PR: https://git.openjdk.java.net/valhalla/pull/370



More information about the valhalla-dev mailing list