On Mon, Jun 19, 2023 at 17:33:58 -0400, Jason Merrill wrote: > On 5/12/23 10:24, Ben Boeckel wrote: > > `file` can be omitted (the `output_stream` will be used then). I *think* > > I see that adding: > > > > %{fdeps_file:-fdeps-file=%{!o:%b.ddi}%{o*:%.ddi%*}} > > %{!fdeps-file: but yes. > > > would at least do for `-fdeps-file` defaults? I don't know if there's a > > reasonable default for `-fdeps-target=` though given that this command > > line has no information about the object file that will be used (`-o` is > > used for preprocessor output since we're leaning on `-E` here). > > I would think it could default to %b.o?
I suppose that could work, yes. > I had quite a few more comments on the v5 patch that you didn't respond > to here or address in the v6 patch; did your mail client hide them from you? Oof. Sorry, I saw large chunks of quoting and apparently assumed the rest was fine (I usually do aggressive trimming when doing that style of review). I see them now. Will go through and include in v7. --Ben