[9] RFR(M): 8130832: Extend the WhiteBox API to provide information about the availability of compiler intrinsics

John Rose john.r.rose at oracle.com
Tue Jul 21 19:02:43 UTC 2015

Yes, that will work, and I think it is cleaner than what we had before, as well as providing the new required functionality.

Reviewed; please get a second reviewer.

— John

P.S. If the unit tests want to test (via the whitebox API) whether an intrinsic was compiled successfully, we might want to expose Compile::gather_intrinsic_statistics, etc.  But not in this change set.

P.P.S. As I think I said before, I wish we had a way to consolidate the switch statements further (into vmSymbols.hpp).  But I don't see a clean way to do it.

On Jul 21, 2015, at 8:19 AM, Zoltán Majó <zoltan.majo at oracle.com> wrote:
> Here is the newest webrev:
> - top: http://cr.openjdk.java.net/~zmajo/8130832/top/ <http://cr.openjdk.java.net/~zmajo/8130832/top/>
> - hotspot: http://cr.openjdk.java.net/~zmajo/8130832/hotspot/webrev.03/ <http://cr.openjdk.java.net/~zmajo/8130832/hotspot/webrev.03/>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20150721/19e86c5e/attachment-0001.html>

More information about the hotspot-compiler-dev mailing list