On 12/05/13 14:17, Jan-Benedict Glaw wrote:
Hi!

This patch silences warnings (unused static functions, ambiguous
`else') for the score target.

2013-12-05  Jan-Benedict Glaw  <jbg...@lug-owl.de>

        * config/score/score.c (score_force_temporary): Delete function.
        (score_split_symbol): Ditto.
        * config/score/score.h (ASM_OUTPUT_ADDR_DIFF_ELT): Add extra
        parentheses to silence ambiguity warning.

[snip]

diff --git a/gcc/config/score/score.h b/gcc/config/score/score.h
index 5ab7875..e3bb7f0 100644
--- a/gcc/config/score/score.h
+++ b/gcc/config/score/score.h
@@ -757,11 +757,12 @@ typedef struct score_args
     entries PC-relative in GP-relative when .gp(d)word is supported.  */
  #define ASM_OUTPUT_ADDR_DIFF_ELT(STREAM, BODY, VALUE, REL)                \
    do {                                                                    \
-    if (TARGET_SCORE7)                                                    \
+    if (TARGET_SCORE7) {                                                  \
        if (flag_pic)                                                       \
          fprintf (STREAM, "\t.gpword %sL%d\n", LOCAL_LABEL_PREFIX, VALUE); \
        else                                                                \
          fprintf (STREAM, "\t.word %sL%d\n", LOCAL_LABEL_PREFIX, VALUE);   \
+    }                                                                     \
    } while (0)

  /* Jump table alignment is explicit in ASM_OUTPUT_CASE_LABEL.  */
Formatting nit. The open curley should be on its own line, indented 2 spaces relative to its conditional. Its close paren should also be on its own line at the same indention level as the open curley.

Note this will cause all the code inside the new curleys to need to be reindented as well.

jeff

Reply via email to