Hi Andrew,
Please take a look at this updated patch:
http://cr.openjdk.java.net/~ngasson/8209413/webrev.03/
It contains your suggested change below, as well as an extra check in
the AARCH64Frame two-argument constructor that the PC from SP[-1] is
valid. Also changed SharedRuntime::generate_nat
Dear All,
Thanks a lot for your reviewing.
Here I refined the patch to fix a minor bug that cause BasicJMapTest
fail with jmap -histo:all.
May I ask your help again to review it?
http://cr.openjdk.java.net/~xiaofeya/8215622/webrev.09/
Thanks!
Cheers,
Lin