This revision was automatically updated to reflect the committed changes.
Closed by commit rL328196: [CodeGen] Emit DWARF constructor calling
convention (authored by JDevlieghere, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
aprantl accepted this revision.
aprantl added a comment.
This revision is now accepted and ready to land.
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.
aguinet updated this revision to Diff 133745.
aguinet added a comment.
Herald added a subscriber: krytarowski.
Test case added!
https://reviews.llvm.org/D42351
Files:
lib/CodeGen/CGDebugInfo.cpp
test/CodeGen/debug-info-cc.c
Index: test/CodeGen/debug-info-cc.c
probinson added a comment.
+1 for testcase.
Repository:
rC Clang
https://reviews.llvm.org/D42351
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aprantl added a comment.
If we take https://reviews.llvm.org/D42350, taking this is fairly obvious,
though it could use a testcase.
Repository:
rC Clang
https://reviews.llvm.org/D42351
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
rnk added a subscriber: aprantl.
rnk added a comment.
@aprantl
Repository:
rC Clang
https://reviews.llvm.org/D42351
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aguinet created this revision.
aguinet added a reviewer: rnk.
Herald added subscribers: cfe-commits, JDevlieghere.
This needs https://reviews.llvm.org/D42350 in LLVM!
Repository:
rC Clang
https://reviews.llvm.org/D42351
Files:
lib/CodeGen/CGDebugInfo.cpp
Index: