[gcc r14-8404] RISC-V: Don't make Ztso imply A

Patrick O'Neill poneill@gcc.gnu.org
Thu Jan 25 00:37:22 GMT 2024


https://gcc.gnu.org/g:0d1b0f2ad391217a43e9239ca017958f89e8f076

commit r14-8404-g0d1b0f2ad391217a43e9239ca017958f89e8f076
Author: Palmer Dabbelt <palmer@rivosinc.com>
Date:   Tue Dec 12 19:54:05 2023 -0800

    RISC-V: Don't make Ztso imply A
    
    I can't actually find anything in the ISA manual that makes Ztso imply
    A.  In theory the memory ordering is just a different thing that the set
    of availiable instructions (ie, Ztso without A would still imply TSO for
    loads and stores).  It also seems like a configuration that could be
    sane to build: without A it's all but impossible to write any meaningful
    multi-core code, and TSO is really cheap for a single core.
    
    That said, I think it's kind of reasonable to provide A to users asking
    for Ztso.  So maybe even if this was a mistake it's the right thing to
    do?
    
    gcc/ChangeLog:
    
            * common/config/riscv/riscv-common.cc (riscv_implied_info):
            Remove {"ztso", "a"}.

Diff:
---
 gcc/common/config/riscv/riscv-common.cc | 2 --
 1 file changed, 2 deletions(-)

diff --git a/gcc/common/config/riscv/riscv-common.cc b/gcc/common/config/riscv/riscv-common.cc
index 4da7f37303b..0a3c5cdfae5 100644
--- a/gcc/common/config/riscv/riscv-common.cc
+++ b/gcc/common/config/riscv/riscv-common.cc
@@ -96,8 +96,6 @@ static const riscv_implied_info_t riscv_implied_info[] =
   {"zks", "zksed"},
   {"zks", "zksh"},
 
-  {"ztso", "a"},
-
   {"v", "zvl128b"},
   {"v", "zve64d"},


More information about the Gcc-cvs mailing list