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/builtins.def | |
| 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/builtins.def')
| -rw-r--r-- | gcc/builtins.def | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/gcc/builtins.def b/gcc/builtins.def index 076da40..ed850df 100644 --- a/gcc/builtins.def +++ b/gcc/builtins.def @@ -945,3 +945,6 @@ DEF_GCC_BUILTIN (BUILT_IN_LINE, "LINE", BT_FN_INT, ATTR_NOTHROW_LEAF_LIST) /* Pointer Bounds Checker builtins. */ #include "chkp-builtins.def" + +#undef DEF_BUILTIN_CHKP +#undef DEF_BUILTIN |