[2/3,ARC] Remove malign-call

Message ID 20200303094046.1439895-2-claziss@gmail.com
State New
Headers show
Series
  • [1/3,ARC] Remove mmixed-code option.
Related show

Commit Message

Claudiu Zissulescu March 3, 2020, 9:40 a.m.
The malign-call option is obsolete, remove it.

gcc/
xxxx-xx-xx  Claudiu Zissulescu  <claziss@synopsys.com>

	* config/arc/arc.opt (malign-call): Remove option.
	* doc/invoke.texi (ARC): Remove malign-call doc.
	* common/config/arc/arc-common.c (arc_option_optimization_table):
	Remove malign-call.
---
 gcc/common/config/arc/arc-common.c | 1 -
 gcc/config/arc/arc.opt             | 4 ----
 gcc/doc/invoke.texi                | 6 +-----
 3 files changed, 1 insertion(+), 10 deletions(-)

-- 
2.24.1

Comments

Jeff Law March 3, 2020, 5:01 p.m. | #1
On Tue, 2020-03-03 at 11:40 +0200, Claudiu Zissulescu wrote:
> The malign-call option is obsolete, remove it.

> 

> gcc/

> xxxx-xx-xx  Claudiu Zissulescu  <claziss@synopsys.com>

> 

> 	* config/arc/arc.opt (malign-call): Remove option.

> 	* doc/invoke.texi (ARC): Remove malign-call doc.

> 	* common/config/arc/arc-common.c (arc_option_optimization_table):

> 	Remove malign-call.

Similarly.  No problem removing the code, but standard operating procedure is
to leave the option.  Also applies to patch 3/3 of this series.

jeff
>

Patch

diff --git a/gcc/common/config/arc/arc-common.c b/gcc/common/config/arc/arc-common.c
index 14c20123c70..7f46f547e30 100644
--- a/gcc/common/config/arc/arc-common.c
+++ b/gcc/common/config/arc/arc-common.c
@@ -62,7 +62,6 @@  static const struct default_options arc_option_optimization_table[] =
     { OPT_LEVELS_SIZE, OPT_fif_conversion, NULL, 0 },
     { OPT_LEVELS_1_PLUS, OPT_fomit_frame_pointer, NULL, 1 },
     { OPT_LEVELS_3_PLUS_SPEED_ONLY, OPT_msize_level_, NULL, 0 },
-    { OPT_LEVELS_3_PLUS_SPEED_ONLY, OPT_malign_call, NULL, 1 },
     { OPT_LEVELS_NONE, 0, NULL, 0 }
   };
 
diff --git a/gcc/config/arc/arc.opt b/gcc/config/arc/arc.opt
index 2b2b947ca08..a8af5138183 100644
--- a/gcc/config/arc/arc.opt
+++ b/gcc/config/arc/arc.opt
@@ -301,10 +301,6 @@  mannotate-align
 Target Var(TARGET_ANNOTATE_ALIGN)
 Explain what alignment considerations lead to the decision to make an insn short or long.
 
-malign-call
-Target Var(TARGET_ALIGN_CALL)
-Do alignment optimizations for call instructions.
-
 mRcq
 Target Var(TARGET_Rcq)
 Enable Rcq constraint handling - most short code generation depends on this.
diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index 7627e889b5d..802d36d4098 100644
--- a/gcc/doc/invoke.texi
+++ b/gcc/doc/invoke.texi
@@ -723,7 +723,7 @@  Objective-C and Objective-C++ Dialects}.
 -mlong-calls  -mmedium-calls  -msdata  -mirq-ctrl-saved @gol
 -mrgf-banked-regs  -mlpc-width=@var{width}  -G @var{num} @gol
 -mvolatile-cache  -mtp-regno=@var{regno} @gol
--malign-call  -mauto-modify-reg  -mbbit-peephole  -mno-brcc @gol
+-mauto-modify-reg  -mbbit-peephole  -mno-brcc @gol
 -mcase-vector-pcrel  -mcompact-casesi  -mno-cond-exec  -mearly-cbranchsi @gol
 -mexpand-adddi  -mindexed-loads  -mlra  -mlra-priority-none @gol
 -mlra-priority-compact mlra-priority-noncompact  -mmillicode @gol
@@ -17861,10 +17861,6 @@  Enable cache bypass for volatile references.
 The following options fine tune code generation:
 @c code generation tuning options
 @table @gcctabopt
-@item -malign-call
-@opindex malign-call
-Do alignment optimizations for call instructions.
-
 @item -mauto-modify-reg
 @opindex mauto-modify-reg
 Enable the use of pre/post modify with register displacement.