Re: [PATCH] Add -gcodeview option

2022-10-26 Thread Mark Harmstone
On 25/10/22 09:21, Martin Storsjö wrote: Btw, stylistically, should we strive towards using double dashes for the pdb option? I think that's the most canonical way for such getopt options (even though it doesn't make any practical difference). I've started trying to update various users to pref

Re: [PATCH] Add -gcodeview option

2022-10-25 Thread Martin Storsjö
On Tue, 25 Oct 2022, Mark Harmstone wrote: On 24/10/22 12:08, Martin Storsjö wrote: Hmm, what does this end up passing to the linker in the end - does it just pass "-pdb="? (What does the "*" parameter do here?) If that's the case - that sounds reasonable - assuming that if a user passes an ex

Re: [PATCH] Add -gcodeview option

2022-10-24 Thread Mark Harmstone
On 24/10/22 12:08, Martin Storsjö wrote: Hmm, what does this end up passing to the linker in the end - does it just pass "-pdb="? (What does the "*" parameter do here?) If that's the case - that sounds reasonable - assuming that if a user passes an extra -Wl,--pdb,myspecificname.pdb, that would ta

Re: [PATCH] Add -gcodeview option

2022-10-24 Thread Martin Storsjö
On Mon, 24 Oct 2022, Mark Harmstone wrote: Both current lld and the next version of ld have an option -pdb, which creates a PDB file which Microsoft's debuggers can use. This patch adds a -gcodeview option, which passes this to the linker. I do intend to expand this so it also creates the .debu