Enable debugging with LLDB (Xcode, Qt Creator, etc) (hsutter#744)

* Print line directives before function definitions

Line directive filenames now have absolute paths.
These changes are required to enable debugging with LLDB.

* Add command line option to enable absolute paths in line directives (defaults to off)

Author: bluetarpmedia

source/cppfront.cpp

@@ -19,6 +19,7 @@
 #include <iostream>
 #include <cstdio>
 #include <optional>
+#include <filesystem>
 
 namespace cpp2 {
 
@@ -68,6 +69,14 @@ static cmdline_processor::register_flag cmd_noline(
     []{ flag_clean_cpp1 = true; }
 );
 
+static auto flag_absolute_line_directives = false;
+static cmdline_processor::register_flag cmd_absolute_line_directives(
+    9,
+    "absolute-line-directives",
+    "Emit absolute paths in #line directives",
+    [] { flag_absolute_line_directives = true; }
+);
+
 static auto flag_import_std = false;
 static cmdline_processor::register_flag cmd_import_std(
     0,
@@ -222,6 +231,11 @@ class positional_printer
     bool            printed_extra               = false;
     char            last_printed_char           = {};
 
+    struct line_directive_info {
+        lineno_t out_pos_line;                        // output lineno where the line directive was printed
+        lineno_t line;                                // the lineno in the line directive
+    } prev_line_directive = {};
+
     struct req_act_info {
         colno_t requested;
         colno_t offset;
@@ -401,6 +415,15 @@ class positional_printer
             return;
         }
 
+        //  Don't print duplicate line directives on subsequent lines
+        if (
+            prev_line_directive.out_pos_line == curr_pos.lineno
+            && prev_line_directive.line == line
+            )
+        {
+            return;
+        }
+
         //  Otherwise, implement the request
         prev_line_info = { curr_pos.lineno, { } };
         ensure_at_start_of_new_line();
@@ -411,6 +434,7 @@ class positional_printer
             *out << "#line " << line << " " << std::quoted(cpp2_filename) << "\n";
         }
         just_printed_line_directive = true;
+        prev_line_directive = { curr_pos.lineno, line };
     }
 
     //  Catch up with comment/blank lines
@@ -620,7 +644,9 @@ class positional_printer
     )
         -> void
     {
-        cpp2_filename = cpp2_filename_;
+        cpp2_filename = (flag_absolute_line_directives) ?
+            std::filesystem::absolute(std::filesystem::path(cpp2_filename_)).string() :
+            cpp2_filename_;
         assert(
             !is_open()
             && !pcomments
@@ -5601,6 +5627,12 @@ class cppfront
                 )
             )
         {
+            //  Print a line directive before every function definition
+            //  (needed to enable debugging with lldb).
+            if (n.initializer) {
+                printer.print_extra("");
+            }
+
             auto is_streaming_operator = [](std::string_view sv) {
                 return
                     sv == "operator<<"
@@ -5932,7 +5964,6 @@ class cppfront
             //  Else emit the definition
             else if (n.initializer)
             {
-
                 if (func->returns.index() == function_type_node::list) {
                     auto& r = std::get<function_type_node::list>(func->returns);
                     function_returns.emplace_back(r.get());