On Wed, 2024-01-03 at 00:41 +0000, Joseph Myers wrote: > On Tue, 2 Jan 2024, David Malcolm wrote: > > > > > + #print(f'{url_suffix=} {index_text=}') > > > > > > Various commented-out or "if 0" debugging code like this should > > > probably > > > be removed (or made into an actual runtime conditional if > > > desired). > > > > I've removed them all. > > There are still a few left.
Oops; sorry about that. > The patch is OK with the remaining > commented-out code removed or made into runtime conditionals. Thanks. I removed the commented-out code, rebased, reran the unit tests, regenerated the .opt.urls files, and retested the resulting patch kit successfully (on x86_64-pc-linux-gnu with all langs+jit enabled). I've now pushed the patch kit to trunk as these four commits: options: add gcc/regenerate-opt-urls.py: r14-6920-g9e49746da303b8 Add generated .opt.urls files: r14-6921-g5bb18475a10d05 opts: add logic to generate options-urls.cc: r14-6922-g6ecc1e32353e33 options: wire up options-urls.cc into gcc_urlifier: r14-6923-g4ded42c2c5a5c9 Thanks Dave