review for 7061101: adlc should complain about mixing block and expression forms of ins_encode

Tom Rodriguez tom.rodriguez at oracle.com
Thu Jun 30 12:50:53 PDT 2011


How about:

+  // Check for duplicate ins_encode sections after parsing the block                                                                                
+  // so that parsing can continue and find any other errors. 

tom

On Jun 30, 2011, at 11:27 AM, Vladimir Kozlov wrote:

> Yes, could it says something like this?:
> 
> // We do this check after block parsing so that we can also catch problems in the block
> 
> Vladimir
> 
> Tom Rodriguez wrote:
>> On Jun 30, 2011, at 11:20 AM, Vladimir Kozlov wrote:
>>> Looks good but I do not understand your comment. Did you mean the check is done after parsing a ins_encode block instead of before it?
>> Yes.  Should I change it somehow?
>> tom
>>> Thanks,
>>> Vladimir
>>> 
>>> Tom Rodriguez wrote:
>>>> http://cr.openjdk.java.net/~never/7061101
>>>> 12 lines changed: 12 ins; 0 del; 0 mod; 5050 unchg
>>>> 7061101: adlc should complain about mixing block and expression forms of ins_encode
>>>> Reviewed-by:
>>>> Only one ins_encode should be specified.  Tested with bad ad file.



More information about the hotspot-compiler-dev mailing list