> On May 30, 2019, at 2:13 PM, Bernhard Reutner-Fischer <rep.dot....@gmail.com> > wrote: > > On 30 May 2019 18:23:43 CEST, Qing Zhao <qing.z...@oracle.com> wrote: >> Hi, >> >> PR 90581 (provide an option to adjust the maximum depth of nested >> #include) >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90581 >> >> is to add a new cpp option -fmax-inlcude-depth > > Typo inl vs inc.
Okay, will fix this. > > Why isn't this a param? do you mean to change “-fmax-include-depth=” to “-param max-include-depth=“? > Wouldn't a param ease range checking not to overflow the uint max and maybe > automagically provide diagnostics for out of range input? > > In the docs you mix "number" and "depth”. will fix this. thanks for the feedback. Qing > > thanks, > >> to set the maximum depth >> of nested #include. >> >> '-fmax-include-depth=DEPTH' >> Set the maximum depth of the nested include. The default value is >> 200. >> >> Please check the attached patch. >> I have done bootstrap and regression test on X86, no any issue. >> >> thanks a lot. >> >> Qing. >> >> >> >> gcc/ChangeLog: >> >> 2019-05-30 qing zhao <qing.z...@oracle.com> >> >> * doc/cppopts.texi: Add document for -fmax-include-depth. >> * doc/invoke.texi (Preprocessor Options): List -fmax-include-depth. >> >> libcpp/ChangeLog: >> >> 2019-05-30 qing zhao <qing.z...@oracle.com> >> >> * directives.c (do_include_common): Replace CPP_STACK_MAX with >> CPP_OPTION (pfile, max_include_depth). >> * include/cpplib.h (struct cpp_options): Add new field >> max_include_depth. >> * init.c (cpp_create_reader): Initiate new field max_include_depth. >> * internal.h: Delete CPP_STACK_MAX. >> >> gcc/c-family/ChangeLog: >> >> 2019-05-30 qing zhao <qing.z...@oracle.com> >> >> * c-opts.c (c_common_handle_option): Handle -fmax-include-depth. >> * c.opt: Add new option -fmax-include-depth. >> >> gcc/testsuite/ChangeLog: >> >> 2019-05-30 qing zhao <qing.z...@oracle.com> >> >> * c-c++-common/cpp/fmax-include-depth-1a.h: New test. >> * c-c++-common/cpp/fmax-include-depth-1b.h: New test. >> * c-c++-common/cpp/fmax-include-depth.c: New test. >
