[gcc r14-8208] opts: Fix up -ffold-mem-offsets option keywords

Jakub Jelinek jakub@gcc.gnu.org
Thu Jan 18 07:45:41 GMT 2024


https://gcc.gnu.org/g:1203fc2e6a40c65896763554f62cacfb4bd6a836

commit r14-8208-g1203fc2e6a40c65896763554f62cacfb4bd6a836
Author: Jakub Jelinek <jakub@redhat.com>
Date:   Thu Jan 18 08:45:09 2024 +0100

    opts: Fix up -ffold-mem-offsets option keywords
    
    While the option was originally meant to be a Target option for a single
    target, it is an option for all targets, so should be Common rather than
    Target, and because it is an optimization option which could be different
    in between different LTO TUs, I've added Optimization keyword too.
    From what I can see, Bool is a non-documented non-existing keyword (at
    least, grep Bool *.awk shows nothing, so I've dropped that too.  Seems
    that the option parsing simply parses and ignores any non-existing keywords.
    
    Guess we should drop the Bool keywords from the gcc/config/riscv/riscv.opt
    file eventually, so that people don't copy this around.
    
    2024-01-18  Jakub Jelinek  <jakub@redhat.com>
    
            PR other/113399
            * common.opt (ffold-mem-offsets): Remove Target and Bool keywords, add
            Common and Optimization.

Diff:
---
 gcc/common.opt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/common.opt b/gcc/common.opt
index 5f0a101bccb..7dc380cc153 100644
--- a/gcc/common.opt
+++ b/gcc/common.opt
@@ -1262,7 +1262,7 @@ Common Var(flag_cprop_registers) Optimization
 Perform a register copy-propagation optimization pass.
 
 ffold-mem-offsets
-Target Bool Var(flag_fold_mem_offsets) Init(1)
+Common Var(flag_fold_mem_offsets) Init(1) Optimization
 Fold instructions calculating memory offsets to the memory access instruction if possible.
 
 fcrossjumping


More information about the Gcc-cvs mailing list