RFR: 8139589: [JVMCI] throw exceptions in faulty code installation operations

[Roland Schatz]

New webrev, including tests:
http://cr.openjdk.java.net/~rschatz/JDK-8139589/webrev.01/

All of these unit tests crash the VM without these changes. With these 
changes, they simply throw an exception.

This also includes a few more error checks compared to the previous 
webrev. These are mainly null accesses that were missed previously 
because there was no explicit "assert" or "guarantee" check in the code.

- Roland


On 11/05/2015 07:27 PM, Christian Thalinger wrote:
>
>> On Nov 5, 2015, at 2:46 AM, Roland Schatz <roland.schatz at oracle.com 
>> <mailto:roland.schatz at oracle.com>> wrote:
>>
>> Hi,
>>
>> That's a followup change to the JVMCI integration. Please review.
>>
>> JIRA:
>> https://bugs.openjdk.java.net/browse/JDK-8139589
>>
>> Webrev:
>> http://cr.openjdk.java.net/~rschatz/JDK-8139589/webrev.00/ 
>> <http://cr.openjdk.java.net/%7Erschatz/JDK-8139589/webrev.00/>
>
> I believe you wrote some tests for these changes.  Tom said some use 
> Graal functionally and some don’t.  Could you add the ones that don’t 
> to this change?
>
>>
>> Thanks,
>> Roland
>

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