[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2018-12-03 Thread iii at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 --- Comment #16 from iii at gcc dot gnu.org --- Author: iii Date: Mon Dec 3 09:49:02 2018 New Revision: 266734 URL: https://gcc.gnu.org/viewcvs?rev=266734&root=gcc&view=rev Log: Repeat jump threading after combine Consider the following RTL: (

[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2018-09-24 Thread iii at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 --- Comment #15 from iii at gcc dot gnu.org --- Author: iii Date: Mon Sep 24 14:21:03 2018 New Revision: 264535 URL: https://gcc.gnu.org/viewcvs?rev=264535&root=gcc&view=rev Log: S/390: Fix conditional returns on z196+ S/390 epilogue ends with (

[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2018-09-06 Thread krebbel at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 --- Comment #14 from Andreas Krebbel --- Author: krebbel Date: Thu Sep 6 07:38:42 2018 New Revision: 264143 URL: https://gcc.gnu.org/viewcvs?rev=264143&root=gcc&view=rev Log: S/390: Prohibit SYMBOL_REF in UNSPECV_CAS Inhibit constant propagati

[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2018-09-06 Thread krebbel at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 --- Comment #13 from Andreas Krebbel --- Author: krebbel Date: Thu Sep 6 07:35:35 2018 New Revision: 264142 URL: https://gcc.gnu.org/viewcvs?rev=264142&root=gcc&view=rev Log: S/390: Register pass_s390_early_mach statically The dump file used t

[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2018-08-22 Thread iii at linux dot ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 --- Comment #12 from Ilya Leoshkevich --- I've investigated foo3, foo4 and foo5, and came to the following conclusions: When foo3 is compiled with -march=z10 or later, cprop1 pass propagates global's SYMBOL_REF value into UNSPECV_CAS. On previo

[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2018-08-06 Thread stli at linux dot ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 --- Comment #11 from stli at linux dot ibm.com --- Hi, I've retested the samples with gcc 7, 8 and head from 2018-07-20, but there are still issues: The examples foo1 and foo2 are okay. The issue in example foo3 is still present (see description

[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2017-04-25 Thread krebbel at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 --- Comment #10 from Andreas Krebbel --- Author: krebbel Date: Tue Apr 25 11:11:48 2017 New Revision: 247189 URL: https://gcc.gnu.org/viewcvs?rev=247189&root=gcc&view=rev Log: S/390: PR80080: Optimize atomic patterns. The attached patch optimiz

[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2017-04-25 Thread krebbel at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 --- Comment #9 from Andreas Krebbel --- Author: krebbel Date: Tue Apr 25 07:37:50 2017 New Revision: 247132 URL: https://gcc.gnu.org/viewcvs?rev=247132&root=gcc&view=rev Log: S/390: PR80080: Optimize atomic patterns. The attached patch optimize

[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2017-03-22 Thread vogt at linux dot vnet.ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 --- Comment #8 from Dominik Vogt --- The patch has a performance regression on s390x. .L1 lr %r3,%r1 cs %r1,%r4,0(%r2) jne .L1 becomes .L1 cs %r1,%r3,0(%r2) ipm %r4 sra %r4,28 cijne %r4,0,.L1 Alth

[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2017-03-21 Thread vogt at linux dot vnet.ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 --- Comment #7 from Dominik Vogt --- (In reply to Jakub Jelinek from comment #6) > I think it depends on what > (success, old_reg) = compare-and-swap(mem, old_reg, new_reg) > sets if success is true. Is there a guarantee that old_reg will be ass

[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2017-03-21 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 --- Comment #6 from Jakub Jelinek --- I think it depends on what (success, old_reg) = compare-and-swap(mem, old_reg, new_reg) sets if success is true. Is there a guarantee that old_reg will be assigned whatever has been passed as the second argu

[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2017-03-21 Thread vogt at linux dot vnet.ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 --- Comment #5 from Dominik Vogt --- What case do you mean? The + if (oldval != old_reg) +emit_move_insn (old_reg, oldval); at the end should make sure that the oldval-rtx is either not changed by the call, or its value is copied into old

[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2017-03-21 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #4

[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2017-03-21 Thread vogt at linux dot vnet.ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 Dominik Vogt changed: What|Removed |Added CC||vogt at linux dot vnet.ibm.com --- Commen

[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2017-03-17 Thread krebbel at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 Andreas Krebbel changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

[Bug target/80080] S390: Isses with emitted cs-instructions for __atomic builtins.

2017-03-17 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80080 Richard Biener changed: What|Removed |Added Keywords||missed-optimization --- Comment #1 from