This is an archive of the discontinued LLVM Phabricator instance.

Emit DWARF "constructor" calling convention for every supported Clang CC
ClosedPublic

Authored by aguinet on Jan 21 2018, 3:50 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

aguinet created this revision.Jan 21 2018, 3:50 AM
aprantl added a subscriber: debug-info.

If we take D42350, taking this is fairly obvious, though it could use a testcase.

+1 for testcase.

aguinet updated this revision to Diff 133745.Feb 10 2018, 2:29 AM

Test case added!

aprantl accepted this revision.Feb 12 2018, 9:03 AM

Looks good with one additional change to the test.

Side note: We should also add all the LLVM extensions to http://wiki.dwarfstd.org/index.php?title=Vendor_Extensions.

test/CodeGen/debug-info-cc.c
1 ↗(On Diff #133745)

I think I would prefer this to be written as

// RUN: $clang_cc1 ... %s | FileCheck --check-prefix=LINUX
// RUN: $clang_cc1 ... %s | FileCheck --check-prefix=WINDOWS
...


// LINUX: !DISubprogram ...
This revision is now accepted and ready to land.Feb 12 2018, 9:03 AM
This revision was automatically updated to reflect the committed changes.