Page MenuHomePhabricator

dcandler (David Candler)
User

Projects

User does not belong to any projects.

User Details

User Since
Jul 31 2019, 5:45 AM (11 w, 5 d)

Recent Activity

Thu, Oct 17

dcandler added a comment to D67216: [cfi] Add flag to always generate .debug_frame.

I added the negative option more as a way to disable the flag, since I'm currently looking at cases where it may want to be turned on by default (and a negative option would then allow you to only get .eh_frame in cases where you'd get both .debug_frame/.eh_frame).

Thu, Oct 17, 9:28 AM · Restricted Project, Restricted Project
dcandler added a comment to D67216: [cfi] Add flag to always generate .debug_frame.

I already spotted the line in CommandFlags.inc needs formatting with a couple of breaks. Also the help text in Options.td could be clearer. In particular, -gno-dwarf-frame shouldn't suggest .debug_frame won't be generated at all (since -g might still emit it). It should probably be more along the lines of:

Thu, Oct 17, 2:35 AM · Restricted Project, Restricted Project

Fri, Oct 11

dcandler committed rL374563: Request commit access for dcandler.
Request commit access for dcandler
Fri, Oct 11, 8:05 AM
dcandler updated the diff for D67216: [cfi] Add flag to always generate .debug_frame.

I've modified the patch so that the new flag will ensure the cfi instructions are actually present to be emitted as well. I went ahead and renamed the flag -gdwarf-frame too, to better reflect that it's dealing with the debug information you'd otherwise get with -g, and is meant to specifically put the information in a .debug_frame section and not .eh_frame.

Fri, Oct 11, 6:22 AM · Restricted Project, Restricted Project

Sep 6 2019

dcandler added a comment to D67216: [cfi] Add flag to always generate .debug_frame.

I was actually torn myself on whether to put the flag in the g group or not, so I'm happy to rename it. As far as I could find, no compiler has an existing option to control this: instead armcc always includes a debug_frame section by default to follow Arm's Dwarf specification. Having it as an option seems more flexible than forcing a different behavior.

Sep 6 2019, 8:12 AM · Restricted Project, Restricted Project

Sep 5 2019

dcandler committed rGa59bffb57695: [ARM] Add support for the s,j,x,N,O inline asm constraints (authored by dcandler).
[ARM] Add support for the s,j,x,N,O inline asm constraints
Sep 5 2019, 8:17 AM
dcandler committed rL371079: [ARM] Add support for the s,j,x,N,O inline asm constraints.
[ARM] Add support for the s,j,x,N,O inline asm constraints
Sep 5 2019, 8:17 AM
dcandler closed D65863: [ARM] Add support for the s,j,x,N,O inline asm constraints.
Sep 5 2019, 8:17 AM · Restricted Project, Restricted Project
dcandler created D67216: [cfi] Add flag to always generate .debug_frame.
Sep 5 2019, 2:22 AM · Restricted Project, Restricted Project

Aug 15 2019

dcandler added a comment to D65863: [ARM] Add support for the s,j,x,N,O inline asm constraints.

Ping. @compnerd any other changes before this could be accepted?

Aug 15 2019, 6:28 AM · Restricted Project, Restricted Project

Aug 8 2019

dcandler updated the diff for D65863: [ARM] Add support for the s,j,x,N,O inline asm constraints.

Adjusted the formatting on some comment lines, and added FIXMEs for all the constraints that require additional validation to clarify what is still needed and where.

Aug 8 2019, 12:10 PM · Restricted Project, Restricted Project
dcandler added inline comments to D65863: [ARM] Add support for the s,j,x,N,O inline asm constraints.
Aug 8 2019, 12:10 PM · Restricted Project, Restricted Project

Aug 7 2019

dcandler created D65863: [ARM] Add support for the s,j,x,N,O inline asm constraints.
Aug 7 2019, 5:13 AM · Restricted Project, Restricted Project

Aug 2 2019

dcandler committed rG7eacefedab6f: [NFC] Test commit, corrected some spelling in comment (authored by dcandler).
[NFC] Test commit, corrected some spelling in comment
Aug 2 2019, 7:45 AM
dcandler committed rL367685: [NFC] Test commit, corrected some spelling in comment.
[NFC] Test commit, corrected some spelling in comment
Aug 2 2019, 7:43 AM
dcandler closed D65516: [NFC] Test commit, corrected some spelling in comment.
Aug 2 2019, 7:43 AM · Restricted Project

Jul 31 2019

dcandler created D65516: [NFC] Test commit, corrected some spelling in comment.
Jul 31 2019, 7:58 AM · Restricted Project