| Message ID | CAELXzTMtNhb6R1DpztiRodPsQEeZEjJBe1S+YkLKii72rTkUrQ@mail.gmail.com |
|---|---|
| State | New |
| Headers | show |
On Tue, May 17, 2016 at 12:14:28PM +1000, Kugan Vivekanandarajah wrote: > Hi, > > static variable all_extensions in aarch64.c is not used and therefore > dead. I don’t see any reason why it should be there. Attached patch > removes this. > > > Bootstrapped on aarch64-linux-gnu. Regression testing is ongoing. > > Is this OK for trunk? OK. Thanks, James > 2016-05-17 Kugan Vivekanandarajah <kuganv@linaro.org> > > * config/aarch64/aarch64.c (all_extensions): Removed unused static variable.
diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c index e081b16..00ab85b 100644 --- a/gcc/config/aarch64/aarch64.c +++ b/gcc/config/aarch64/aarch64.c @@ -663,16 +663,6 @@ struct aarch64_option_extension const unsigned long flags_off; }; -/* ISA extensions in AArch64. */ -static const struct aarch64_option_extension all_extensions[] = -{ -#define AARCH64_OPT_EXTENSION(NAME, X, FLAGS_ON, FLAGS_OFF, FEATURE_STRING) \ - {NAME, FLAGS_ON, FLAGS_OFF}, -#include "aarch64-option-extensions.def" -#undef AARCH64_OPT_EXTENSION - {NULL, 0, 0} -}; - typedef enum aarch64_cond_code { AARCH64_EQ = 0, AARCH64_NE, AARCH64_CS, AARCH64_CC, AARCH64_MI, AARCH64_PL,
Hi, static variable all_extensions in aarch64.c is not used and therefore dead. I don’t see any reason why it should be there. Attached patch removes this. Bootstrapped on aarch64-linux-gnu. Regression testing is ongoing. Is this OK for trunk? Thanks, Kugan gcc/ChangeLog: 2016-05-17 Kugan Vivekanandarajah <kuganv@linaro.org> * config/aarch64/aarch64.c (all_extensions): Removed unused static variable.