diff options
| author | Richard Sandiford <richard.sandiford@arm.com> | 2015-11-07 10:18:38 +0000 |
|---|---|---|
| committer | Richard Sandiford <rsandifo@gcc.gnu.org> | 2015-11-07 10:18:38 +0000 |
| commit | 0aad01985747ab503fd71ccf4767d0069fc3e85a (patch) | |
| tree | ccf3e9b48b09616d6c7845a4b402ae67fa434d63 /gcc/genmatch.c | |
| parent | 46e343b8d225732ae2e81509ca1accf47444f64b (diff) | |
| download | gcc-0aad01985747ab503fd71ccf4767d0069fc3e85a.zip gcc-0aad01985747ab503fd71ccf4767d0069fc3e85a.tar.gz gcc-0aad01985747ab503fd71ccf4767d0069fc3e85a.tar.bz2 | |
Move #undef DEF_BUILTIN* to builtins.def
I was confused at first why tree-core.h was undefining DEF_BUILTIN_CHKP
before defining it, then undefining it again after including builtins.def.
This is because builtins.def provides a default definition of
DEF_BUILTIN_CHKP, but leaves it up to the caller to undefine it where
necessary. Similarly to the previous internal-fn.def patch, it seems
more obvious for builtins.def to #undef things unconditionally.
One argument might have been that keeping preprocessor stuff
out of the .def files makes it easier for non-cpp parsers. In practice
though we already have #ifs and multiline #defines, so single-line #undefs
should be easy in comparison.
Tested on x86_64-linux-gnu, aarch64-linux-gnu and arm-linux-gnueabi.
gcc/ada/
* gcc-interface/utils.c: Don't undef DEF_BUILTIN.
gcc/c-family/
* c-common.c: Don't undef DEF_BUILTIN.
gcc/jit/
* jit-builtins.c: Don't undef DEF_BUILTIN.
gcc/lto/
* lto-lang.c: Don't undef DEF_BUILTIN.
gcc/
* builtins.def: #undef DEF_BUILTIN and DEF_BUILTIN_CHKP
* builtins.c, genmatch.c, tree-core.h: Don't undef them here.
From-SVN: r229924
Diffstat (limited to 'gcc/genmatch.c')
| -rw-r--r-- | gcc/genmatch.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/gcc/genmatch.c b/gcc/genmatch.c index ad502f9..1eb8c24 100644 --- a/gcc/genmatch.c +++ b/gcc/genmatch.c @@ -229,7 +229,6 @@ enum built_in_function { #include "builtins.def" END_BUILTINS }; -#undef DEF_BUILTIN /* Return true if CODE represents a commutative tree code. Otherwise return false. */ @@ -4619,7 +4618,6 @@ add_operator (VIEW_CONVERT2, "VIEW_CONVERT2", "tcc_unary", 1); #define DEF_BUILTIN(ENUM, N, C, T, LT, B, F, NA, AT, IM, COND) \ add_builtin (ENUM, # ENUM); #include "builtins.def" -#undef DEF_BUILTIN /* Parse ahead! */ parser p (r); |