Problems with vdefault/vwithfield

Tobias Hartmann tobias.hartmann at
Mon Feb 6 13:00:45 UTC 2017


Mr. Simms pointed out that in GenerateOopMap::interp1() me may have to pop arguments for vwithdefault. But then we would need to do the same with vnew. I'm not too familiar with that code, Fred, could you have a look?


On 06.02.2017 13:39, Tobias Hartmann wrote:
> Hi,
> please review the following fix:
> We hit a ShouldNotReachHere in GenerateOopMap::interp1() with "unexpected opcode: 215" because the _vwithfield bytecode is not known. I fixed the same problem in MethodLiveness::BasicBlock::compute_gen_kill_single() and BCEscapeAnalyzer::iterate_one_block() and modified the tests to trigger this.
> Thanks,
> Tobias

More information about the valhalla-dev mailing list