RFR: 8255553: [PPC64] Introduce and use setbc and setnbc P10 instructions [v6]
    Ziviani 
    github.com+670087+jrziviani at openjdk.java.net
       
    Thu Nov  5 12:34:56 UTC 2020
    
    
  
On Thu, 5 Nov 2020 10:50:10 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:
>> Ziviani has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR.
>
> src/hotspot/cpu/ppc/ppc.ad line 11422:
> 
>> 11420: 
>> 11421: // Manifest a CmpL3 result in an integer register.
>> 11422: instruct cmpL3_reg_reg(iRegIdst dst, iRegLsrc src1, iRegLsrc src2) %{
> 
> I had forgotten one detail in my previous review. Sorry for that. We need to model the CR0 effect:
> instruct cmpL3_reg_reg(iRegIdst dst, iRegLsrc src1, iRegLsrc src2, flagsRegCR0 cr0) %{
>   match(Set dst (CmpL3 src1 src2));
>   effect(KILL cr0);
> (Same for other nodes.)
Wow, thanks for catching it. But, let me make my naive question: why is it necessary?
-------------
PR: https://git.openjdk.java.net/jdk/pull/907
    
    
More information about the hotspot-dev
mailing list