These were pretty obvious when looking at the code. Verified the bfin ports from config-list.mk will build with a trunk compiler.

Installing on the trunk. Now onward to the target independent bits (which are few).

Jeff
commit 71012b1c342ab2b69494429ec2d60d94248acea5
Author: law <law@138bc75d-0d04-0410-961f-82ee72b054a4>
Date:   Fri Oct 28 15:22:28 2016 +0000

        * config/bfin/bfin.c (bfin_legitimate_address_p): Add missing
        fallthru comment.
        * config/bfin/bfin.h (TARGET_CPU_CPP_BUILTINS): Likewise.
    
    git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@241651 
138bc75d-0d04-0410-961f-82ee72b054a4

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index cfd0929..3f2ea4d 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2016-10-28  Jeff Law  <l...@redhat.com>
+
+       * config/bfin/bfin.c (bfin_legitimate_address_p): Add missing
+       fallthru comment.
+       * config/bfin/bfin.h (TARGET_CPU_CPP_BUILTINS): Likewise.
+
 2016-10-28  Segher Boessenkool  <seg...@kernel.crashing.org>
 
        PR rtl-optimization/78029
diff --git a/gcc/config/bfin/bfin.c b/gcc/config/bfin/bfin.c
index 9b81868..2dfd038 100644
--- a/gcc/config/bfin/bfin.c
+++ b/gcc/config/bfin/bfin.c
@@ -2718,6 +2718,7 @@ bfin_legitimate_address_p (machine_mode mode, rtx x, bool 
strict)
        && REG_P (XEXP (x, 0))
        && bfin_valid_reg_p (REGNO (XEXP (x, 0)), strict, mode, POST_INC))
       return true;
+    break;
   case PRE_DEC:
     if (LEGITIMATE_MODE_FOR_AUTOINC_P (mode)
        && XEXP (x, 0) == stack_pointer_rtx
diff --git a/gcc/config/bfin/bfin.h b/gcc/config/bfin/bfin.h
index a85c8c4..b5f1544 100644
--- a/gcc/config/bfin/bfin.h
+++ b/gcc/config/bfin/bfin.h
@@ -110,30 +110,35 @@
          break;                                \
        case BFIN_CPU_BF542M:                   \
          builtin_define ("__ADSPBF542M__");    \
+         /* FALLTHRU */                        \
        case BFIN_CPU_BF542:                    \
          builtin_define ("__ADSPBF542__");     \
          builtin_define ("__ADSPBF54x__");     \
          break;                                \
        case BFIN_CPU_BF544M:                   \
          builtin_define ("__ADSPBF544M__");    \
+         /* FALLTHRU */                        \
        case BFIN_CPU_BF544:                    \
          builtin_define ("__ADSPBF544__");     \
          builtin_define ("__ADSPBF54x__");     \
          break;                                \
        case BFIN_CPU_BF547M:                   \
          builtin_define ("__ADSPBF547M__");    \
+         /* FALLTHRU */                        \
        case BFIN_CPU_BF547:                    \
          builtin_define ("__ADSPBF547__");     \
          builtin_define ("__ADSPBF54x__");     \
          break;                                \
        case BFIN_CPU_BF548M:                   \
          builtin_define ("__ADSPBF548M__");    \
+         /* FALLTHRU */                        \
        case BFIN_CPU_BF548:                    \
          builtin_define ("__ADSPBF548__");     \
          builtin_define ("__ADSPBF54x__");     \
          break;                                \
        case BFIN_CPU_BF549M:                   \
          builtin_define ("__ADSPBF549M__");    \
+         /* FALLTHRU */                        \
        case BFIN_CPU_BF549:                    \
          builtin_define ("__ADSPBF549__");     \
          builtin_define ("__ADSPBF54x__");     \

Reply via email to