RFR(S) 8010281: Remove code that never execute in opto/ifg.cpp
Niclas Adlertz
niclas.adlertz at oracle.com
Mon Mar 25 02:59:57 PDT 2013
Thank you Vladimir, the new empty lines are now removed.
http://cr.openjdk.java.net/~neliasso/8010281/webrev03
Regards,
Niclas Adlertz
On 22 mar 2013, at 17:16, Vladimir Kozlov wrote:
> Please, remove new empty lines.
>
> Vladimir
>
> On 3/20/13 1:02 AM, Niclas Adlertz wrote:
>> Hi all.
>>
>> Problem:
>> In the C2 register allocator, the define EXACT_PRESSURE is set to decide how we estimate the block pressure. There are currently two different ways on how to estimate this. However, EXACT_PRESSURE is always set to 1. This means that the code for the second choice on how to estimate block pressure is never executed, and therefore not needed.
>>
>> Solution:
>> Remove the define EXACT_PRESSURE and the code that is never executed; executing only the code that now executes within the ifdefs of EXACT_PRESSURE.
>>
>> A review would be much appreciated.
>> Thank you.
>>
>> Regards,
>> Niclas Adlertz
>>
>> ---------------------
>>
>> WEBREV: http://cr.openjdk.java.net/~neliasso/8010281/webrev00/
>> BUG: https://jbs.oracle.com/bugs/browse/JDK-8010281, http://bugs.sun.com/view_bug.do?bug_id= 8010281
>>
More information about the hotspot-compiler-dev
mailing list