[15] RFR(T): 8242090: Remove dead code from c1_LIR

Claes Redestad claes.redestad at oracle.com
Mon Apr 6 10:08:58 UTC 2020


On 2020-04-06 08:10, Tobias Hartmann wrote:
> 
> 
> On 03.04.20 16:15, Claes Redestad wrote:
>> lir_fpop_raw also looked unused, but seems to be used on x86_32 only.
>> I'm not sure it's worth the trouble guarding its use with X86 &&
>> NOT_LP64..?
> 
> I gave it a quick try but I don't think it's worth sprinkling additional #ifdefs into the enum and
> the shared code in c1_LinearScan.cpp. I've simply removed the unused fpop_raw() method:
> http://cr.openjdk.java.net/~thartmann/8242090/webrev.02/

Still looks good (and trivial).

/Claes

> 
> Best regards,
> Tobias
> 


More information about the hotspot-compiler-dev mailing list