aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrew Burgess <andrew.burgess@embecosm.com>2017-02-09 00:34:00 +0000
committerAndrew Burgess <aburgess@gcc.gnu.org>2017-02-09 00:34:00 +0000
commitc3bde35a56de3692c3b61caa0340dc80ee5abfcc (patch)
treef4c0cbf7ef5494a5b18c16e139c14879568096db
parent5896a41da8cb6cd887d9af86d81047e38c3df4f1 (diff)
downloadgcc-c3bde35a56de3692c3b61caa0340dc80ee5abfcc.zip
gcc-c3bde35a56de3692c3b61caa0340dc80ee5abfcc.tar.gz
gcc-c3bde35a56de3692c3b61caa0340dc80ee5abfcc.tar.bz2
gcc/arc: Make arc_selected_cpu global
Currently we only make the base_architecture globally available, this means we can tell if we have selected arc700/archs/etc but it's not possible to tell if the user has selected a specific cpu variant, for example nps400. One problem this causes is, for example, in arc-c.def, if we want to add an __NPS400__ define then we need a flag we can check to determine if this is the right thing to do. In this commit the arc_selected_cpu variable (previously local within arc.c) has been made global. Two other variables arc_base_cpu and arc_selected_arch have been deleted, all of this information can be found within (or through) arc_selected_cpu. All uses of arc_base_cpu and arc_selected_arch have been updated. This commit does not introduce any new defines (like __NPS400__), this is just a restructuring commit. The declaration of arc_selected_cpu has moved into arc-arch.h, in contrast to the declaration of arc_base_cpu which was previously in arc.h. This avoids a compilation issue when building libgcc, as the structure and enums declared in arc-arch.h are not included for libgcc then declaring an arc_selected_cpu (a struct type) in arc.h would result in an unknown struct error. We got away with this for arc_base_cpu as that was an enum type. The declaration of arc_selected_cpu in arc.h could have been wrapped in a '#ifndef IN_LIBGCC2 ... #endif', but it felt neater to simply move the declaration into arc-arch.h. gcc/ChangeLog: * config/arc/arc-arch.h (arc_arch_t): Move unchanged to earlier in file. (arc_cpu_t): Change base_architecture field, arch, to a arc_arc_t pointer, arch_info. (arc_cpu_types): Fill the arch_info field with a pointer into the arc_arch_types table. (arc_selected_cpu): Declare. * config/arc/arc.c (arc_selected_cpu): Make global. (arc_selected_arch): Delete. (arc_base_cpu): Delete. (arc_override_options): Remove references to deleted variables, update access to arch information. (ARC_OPT): Update access to arch information. (ARC_OPTX): Likewise. * config/arc/arc.h (arc_base_cpu): Remove declaration. (TARGET_ARC600): Update access to arch information. (TARGET_ARC601): Likewise. (TARGET_ARC700): Likewise. (TARGET_EM): Likewise. (TARGET_HS): Likewise. * config/arc/driver-arc.c (arc_cpu_to_as): Update access to arch information. From-SVN: r245293
-rw-r--r--gcc/ChangeLog25
-rw-r--r--gcc/config/arc/arc-arch.h50
-rw-r--r--gcc/config/arc/arc.c35
-rw-r--r--gcc/config/arc/arc.h15
-rw-r--r--gcc/config/arc/driver-arc.c2
5 files changed, 74 insertions, 53 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 11c9692..4d8e929 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,28 @@
+2017-02-09 Andrew Burgess <andrew.burgess@embecosm.com>
+
+ * config/arc/arc-arch.h (arc_arch_t): Move unchanged to earlier in
+ file.
+ (arc_cpu_t): Change base_architecture field, arch, to a arc_arc_t
+ pointer, arch_info.
+ (arc_cpu_types): Fill the arch_info field with a pointer into the
+ arc_arch_types table.
+ (arc_selected_cpu): Declare.
+ * config/arc/arc.c (arc_selected_cpu): Make global.
+ (arc_selected_arch): Delete.
+ (arc_base_cpu): Delete.
+ (arc_override_options): Remove references to deleted variables,
+ update access to arch information.
+ (ARC_OPT): Update access to arch information.
+ (ARC_OPTX): Likewise.
+ * config/arc/arc.h (arc_base_cpu): Remove declaration.
+ (TARGET_ARC600): Update access to arch information.
+ (TARGET_ARC601): Likewise.
+ (TARGET_ARC700): Likewise.
+ (TARGET_EM): Likewise.
+ (TARGET_HS): Likewise.
+ * config/arc/driver-arc.c (arc_cpu_to_as): Update access to arch
+ information.
+
2017-02-08 Pat Haugen <pthaugen@us.ibm.com>
PR target/78604
diff --git a/gcc/config/arc/arc-arch.h b/gcc/config/arc/arc-arch.h
index 832338b..2344a4b 100644
--- a/gcc/config/arc/arc-arch.h
+++ b/gcc/config/arc/arc-arch.h
@@ -47,6 +47,23 @@ enum base_architecture
BASE_ARCH_END
};
+/* Architecture specific propoerties. */
+
+typedef struct
+{
+ /* Architecture name. */
+ const char *const name;
+
+ /* Architecture class. */
+ enum base_architecture arch_id;
+
+ /* All allowed flags for this architecture. */
+ const unsigned long long flags;
+
+ /* Default flags for this architecture. It is a subset of
+ FLAGS. */
+ const unsigned long long dflags;
+} arc_arch_t;
/* Tune variants. Needs to match the attr_tune enum. */
@@ -66,7 +83,7 @@ typedef struct
const char *const name;
/* Architecture class. */
- enum base_architecture arch;
+ const arc_arch_t *arch_info;
/* Specific processor type. */
enum processor_type processor;
@@ -76,28 +93,8 @@ typedef struct
/* Tune value. */
enum arc_tune_attr tune;
-} arc_cpu_t;
-
-
-/* Architecture specific propoerties. */
-
-typedef struct
-{
- /* Architecture name. */
- const char *const name;
-
- /* Architecture class. */
- enum base_architecture arch;
-
- /* All allowed flags for this architecture. */
- const unsigned long long flags;
-
- /* Default flags for this architecture. It is a subset of
- FLAGS. */
- const unsigned long long dflags;
-} arc_arch_t;
-
+} arc_cpu_t;
const arc_arch_t arc_arch_types[] =
{
@@ -111,13 +108,16 @@ const arc_arch_t arc_arch_types[] =
const arc_cpu_t arc_cpu_types[] =
{
- {"none", BASE_ARCH_NONE, PROCESSOR_NONE, 0, ARC_TUNE_NONE},
+ {"none", NULL, PROCESSOR_NONE, 0, ARC_TUNE_NONE},
#define ARC_CPU(NAME, ARCH, FLAGS, TUNE) \
- {#NAME, BASE_ARCH_##ARCH, PROCESSOR_##NAME, FLAGS, ARC_TUNE_##TUNE},
+ {#NAME, &arc_arch_types [BASE_ARCH_##ARCH], PROCESSOR_##NAME, FLAGS, ARC_TUNE_##TUNE },
#include "arc-cpus.def"
#undef ARC_CPU
- {NULL, BASE_ARCH_END, PROCESSOR_NONE, 0, ARC_TUNE_NONE}
+ {NULL, NULL, PROCESSOR_NONE, 0, ARC_TUNE_NONE}
};
+/* Currently selected cpu type. */
+extern const arc_cpu_t *arc_selected_cpu;
+
#endif
#endif /* GCC_ARC_ARCH_H */
diff --git a/gcc/config/arc/arc.c b/gcc/config/arc/arc.c
index 9089ef4..8a838f9 100644
--- a/gcc/config/arc/arc.c
+++ b/gcc/config/arc/arc.c
@@ -243,11 +243,8 @@ static bool arc_use_by_pieces_infrastructure_p (unsigned HOST_WIDE_INT,
enum by_pieces_operation op,
bool);
-static const arc_cpu_t *arc_selected_cpu;
-static const arc_arch_t *arc_selected_arch;
-
-/* Global var which sets the current compilation architecture. */
-enum base_architecture arc_base_cpu;
+/* Globally visible information about currently selected cpu. */
+const arc_cpu_t *arc_selected_cpu;
/* Implements target hook vector_mode_supported_p. */
@@ -787,11 +784,9 @@ arc_override_options (void)
/* Set the default cpu options. */
arc_selected_cpu = &arc_cpu_types[(int) arc_cpu];
- arc_selected_arch = &arc_arch_types[(int) arc_selected_cpu->arch];
- arc_base_cpu = arc_selected_arch->arch;
/* Set the architectures. */
- switch (arc_selected_arch->arch)
+ switch (arc_selected_cpu->arch_info->arch_id)
{
case BASE_ARCH_em:
arc_cpu_string = "EM";
@@ -822,16 +817,16 @@ arc_override_options (void)
if ((arc_selected_cpu->flags & CODE) \
&& ((target_flags_explicit & MASK) == 0)) \
target_flags |= MASK; \
- if (arc_selected_arch->dflags & CODE) \
+ if (arc_selected_cpu->arch_info->dflags & CODE) \
target_flags |= MASK; \
} while (0);
-#define ARC_OPTX(NAME, CODE, VAR, VAL, DOC) \
- do { \
- if ((arc_selected_cpu->flags & CODE) \
- && (VAR == DEFAULT_##VAR)) \
- VAR = VAL; \
- if (arc_selected_arch->dflags & CODE) \
- VAR = VAL; \
+#define ARC_OPTX(NAME, CODE, VAR, VAL, DOC) \
+ do { \
+ if ((arc_selected_cpu->flags & CODE) \
+ && (VAR == DEFAULT_##VAR)) \
+ VAR = VAL; \
+ if (arc_selected_cpu->arch_info->dflags & CODE) \
+ VAR = VAL; \
} while (0);
#include "arc-options.def"
@@ -844,18 +839,18 @@ arc_override_options (void)
#define ARC_OPTX(NAME, CODE, VAR, VAL, DOC) \
do { \
if ((VAR == VAL) \
- && (!(arc_selected_arch->flags & CODE))) \
+ && (!(arc_selected_cpu->arch_info->flags & CODE))) \
{ \
error ("%s is not available for %s architecture", \
- DOC, arc_selected_arch->name); \
+ DOC, arc_selected_cpu->arch_info->name); \
} \
} while (0);
#define ARC_OPT(NAME, CODE, MASK, DOC) \
do { \
if ((target_flags & MASK) \
- && (!(arc_selected_arch->flags & CODE))) \
+ && (!(arc_selected_cpu->arch_info->flags & CODE))) \
error ("%s is not available for %s architecture", \
- DOC, arc_selected_arch->name); \
+ DOC, arc_selected_cpu->arch_info->name); \
} while (0);
#include "arc-options.def"
diff --git a/gcc/config/arc/arc.h b/gcc/config/arc/arc.h
index b6464f1..64a3724 100644
--- a/gcc/config/arc/arc.h
+++ b/gcc/config/arc/arc.h
@@ -215,15 +215,16 @@ extern const char *arc_cpu_to_as (int argc, const char **argv);
use conditional execution? */
#define TARGET_AT_DBR_CONDEXEC (!TARGET_ARC700 && !TARGET_V2)
-extern enum base_architecture arc_base_cpu;
-
-#define TARGET_ARC600 ((arc_base_cpu == BASE_ARCH_6xx) \
+#define TARGET_ARC600 ((arc_selected_cpu->arch_info->arch_id \
+ == BASE_ARCH_6xx) \
&& (TARGET_BARREL_SHIFTER))
-#define TARGET_ARC601 ((arc_base_cpu == BASE_ARCH_6xx) \
+#define TARGET_ARC601 ((arc_selected_cpu->arch_info->arch_id \
+ == BASE_ARCH_6xx) \
&& (!TARGET_BARREL_SHIFTER))
-#define TARGET_ARC700 (arc_base_cpu == BASE_ARCH_700)
-#define TARGET_EM (arc_base_cpu == BASE_ARCH_em)
-#define TARGET_HS (arc_base_cpu == BASE_ARCH_hs)
+#define TARGET_ARC700 (arc_selected_cpu->arch_info->arch_id \
+ == BASE_ARCH_700)
+#define TARGET_EM (arc_selected_cpu->arch_info->arch_id == BASE_ARCH_em)
+#define TARGET_HS (arc_selected_cpu->arch_info->arch_id == BASE_ARCH_hs)
#define TARGET_V2 (TARGET_EM || TARGET_HS)
#ifdef ARC_MULTILIB_CPU_DEFAULT
diff --git a/gcc/config/arc/driver-arc.c b/gcc/config/arc/driver-arc.c
index 4b816e6..0e13878 100644
--- a/gcc/config/arc/driver-arc.c
+++ b/gcc/config/arc/driver-arc.c
@@ -47,7 +47,7 @@ arc_cpu_to_as (int argc, const char **argv)
}
}
- switch (arc_selected_cpu->arch)
+ switch (arc_selected_cpu->arch_info->arch_id)
{
case BASE_ARCH_em:
if (arc_selected_cpu->flags & FL_CD)