diff options
author | Lewis Hyatt <lhyatt@gmail.com> | 2023-12-12 17:46:36 -0500 |
---|---|---|
committer | Lewis Hyatt <lhyatt@gmail.com> | 2024-03-14 07:33:02 -0400 |
commit | 942497ad74272e0ef16020d628e471c5f21474b0 (patch) | |
tree | 4eeaf9038cc0ecf82b504ab914d16082fe5d8e7f /libcpp | |
parent | 6c166e55b15894ceb07dcc7b55f900e50e24ec5b (diff) | |
download | gcc-942497ad74272e0ef16020d628e471c5f21474b0.zip gcc-942497ad74272e0ef16020d628e471c5f21474b0.tar.gz gcc-942497ad74272e0ef16020d628e471c5f21474b0.tar.bz2 |
libcpp: Fix macro expansion for argument of __has_include [PR110558]
When the file name for a #include directive is the result of stringifying a
macro argument, libcpp needs to take some care to get the whitespace
correct; in particular stringify_arg() needs to see a CPP_PADDING token
between macro tokens so that it can figure out when to output space between
tokens. The CPP_PADDING tokens are not normally generated when handling a
preprocessor directive, but for #include-like directives, libcpp sets the
state variable pfile->state.directive_wants_padding to TRUE so that the
CPP_PADDING tokens will be output, and then everything works fine for
computed includes.
As the PR points out, things do not work fine for __has_include. Fix that by
setting the state variable the same as is done for #include.
libcpp/ChangeLog:
PR preprocessor/110558
* macro.cc (builtin_has_include): Set
pfile->state.directive_wants_padding prior to lexing the
file name, in case it comes from macro expansion.
gcc/testsuite/ChangeLog:
PR preprocessor/110558
* c-c++-common/cpp/has-include-2.c: New test.
* c-c++-common/cpp/has-include-2.h: New test.
Diffstat (limited to 'libcpp')
-rw-r--r-- | libcpp/macro.cc | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/libcpp/macro.cc b/libcpp/macro.cc index a3d2c15..352eb2e 100644 --- a/libcpp/macro.cc +++ b/libcpp/macro.cc @@ -398,6 +398,8 @@ builtin_has_include (cpp_reader *pfile, cpp_hashnode *op, bool has_next) NODE_NAME (op)); pfile->state.angled_headers = true; + const auto sav_padding = pfile->state.directive_wants_padding; + pfile->state.directive_wants_padding = true; const cpp_token *token = cpp_get_token_no_padding (pfile); bool paren = token->type == CPP_OPEN_PAREN; if (paren) @@ -406,6 +408,7 @@ builtin_has_include (cpp_reader *pfile, cpp_hashnode *op, bool has_next) cpp_error (pfile, CPP_DL_ERROR, "missing '(' before \"%s\" operand", NODE_NAME (op)); pfile->state.angled_headers = false; + pfile->state.directive_wants_padding = sav_padding; bool bracket = token->type != CPP_STRING; char *fname = NULL; |