On Fri, 2019-12-13 at 19:27 +0100, Jakub Jelinek wrote: > On Fri, Dec 13, 2019 at 01:11:05PM -0500, David Malcolm wrote: > > gcc/ChangeLog: > > * builtins.def (BUILT_IN_ANALYZER_BREAK): New builtin. > > (BUILT_IN_ANALYZER_DUMP): New builtin. > > (BUILT_IN_ANALYZER_DUMP_EXPLODED_NODES): New builtin. > > (BUILT_IN_ANALYZER_DUMP_NUM_HEAP_REGIONS): New builtin. > > (BUILT_IN_ANALYZER_DUMP_PATH): New builtin. > > (BUILT_IN_ANALYZER_DUMP_REGION_MODEL): New builtin. > > (BUILT_IN_ANALYZER_EVAL): New builtin. > > Is it a good idea to add further builtins without __builtin_ > prefix (unless required for interoperability etc.)?
I think I can do all of these with just string matching on the fndecl names; would that be preferable to having them as builtins? Dave