RFR(XXS): 8136693: add package statement 'compiler.testlibrary' to CompilerUtils

[Tatiana Pivovarova]

On 09/17/2015 05:30 PM, Tatiana Pivovarova wrote:
>
>
> On 09/17/2015 03:54 PM, Igor Ignatyev wrote:
>> Hi Tatiana,
>>
>> the fix looks good.
>> have you checked that the tests which use this class work fine after 
>> your changes?
> I checked that SegmentedCodeCacheDtraceTest (the only test who used 
> CompilerUtils) compiled and I think this check is enough for such 
> changes. 
You were right. There was some bug. Jtreg couldn't find CompilerUtils in 
a new package. Now I point to correct path @library

new webrev : http://cr.openjdk.java.net/~tpivovarova/8136693/webrev.01/

---
Tatiana
>>
>> Igor
>>
>> On 09/17/2015 03:16 PM, Tatiana Pivovarova wrote:
>>> Hi guys!
>>>
>>> Please review this small enhancement.
>>> JRE : https://bugs.openjdk.java.net/browse/JDK-8136693
>>> webrev : http://cr.openjdk.java.net/~tpivovarova/8136693/webrev.00/
>>>
>>> To fix error 'error: cannot find symbol' 'symbol:   variable
>>> CompilerUtils' it needs to add
>>>
>>> packagecompiler.testlibrary;
>>>
>>> to CompilerUtils class
>>>
>>> ---
>>> Thanks,
>>> Tatiana
>>>
>

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