dblaikie added a comment.

Be best to avoid having to 'fixup' the argabi after the fact - and instead 
build the type with the right argabi from the get-go.



================
Comment at: lib/CodeGen/CGDebugInfo.cpp:955-957
+    // DWARFv5 doesn't specify explicit DW_CC_* value for this case,
+    // thus we just return 'normal' here.
+    return llvm::dwarf::DW_CC_normal;
----------------
Could you file a bug/feature request with the DWARF committee? & maybe we 
could/should use an implementation defined extension value here? (assuming the 
DW_CC set has an implementation defined space - which I really hope it does)


================
Comment at: test/CodeGenCXX/debug-info-type-calling-conventions.cpp:11-12
+
+class UserDtor {
+public:
+  ~UserDtor() {}
----------------
I'd probably make this a struct (and drop the 'public:' section since it'll be 
the default then) - since it doesn't seem to matter here.


https://reviews.llvm.org/D30378



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to