Changeset dee1f89

Timestamp:

Aug 13, 2019, 2:35:54 PM (5 years ago)

Author:

Thierry Delisle <tdelisle@…>

Branches:

ADT, arm-eh, ast-experimental, enum, forall-pointer-decay, jacob/cs343-translation, jenkins-sandbox, master, new-ast, new-ast-unique-expr, pthread-emulation, qualifiedEnum

Children:

1118b8b

Parents:

b3976bf

Message:

Libcfa make can now stop cfa-cpp when starting so a gdb session will be attached to it.
Use make ... gdbwaittarget=FILENAME where the filename is the same as the one printed when building with the silent rules.
cfa-cpp will print the gdb command to attach to it.
You may need to change linux permissions for gdb attach, just Google it.

Files:

• libcfa/src/Makefile.am (modified) (1 diff)
• libcfa/src/Makefile.in (modified) (2 diffs)
• src/main.cc (modified) (9 diffs)

libcfa/src/Makefile.am

@@ -26 +26 @@
 VPATH += :../prelude
 
+gdbwaittarget=""
+
 # AM_CFLAGS for all cfa source
 # AM_CFAFLAGS for only cfa source
 # use -no-include-stdhdr to prevent rebuild cycles
 # The built sources must not depend on the installed headers
-AM_CFAFLAGS = -quiet -in-tree -I$(srcdir)/stdhdr @CONFIG_CFAFLAGS@
+AM_CFAFLAGS = -quiet -in-tree -I$(srcdir)/stdhdr $(if $(findstring ${gdbwaittarget}, ${@}), -XCFA --gdb)  @CONFIG_CFAFLAGS@
 AM_CFLAGS = -g -Wall -Wno-unused-function -fPIC @ARCH_FLAGS@ @CONFIG_CFLAGS@
 AM_CCASFLAGS = -g -Wall -Wno-unused-function @ARCH_FLAGS@ @CONFIG_CFLAGS@

libcfa/src/Makefile.in

@@ -435 +435 @@
 am__v_UPP_1 = 
 lib_LTLIBRARIES = libcfa.la libcfathread.la
+gdbwaittarget = ""
 
 # AM_CFLAGS for all cfa source
@@ -440 +441 @@
 # use -no-include-stdhdr to prevent rebuild cycles
 # The built sources must not depend on the installed headers
-AM_CFAFLAGS = -quiet -in-tree -I$(srcdir)/stdhdr @CONFIG_CFAFLAGS@
+AM_CFAFLAGS = -quiet -in-tree -I$(srcdir)/stdhdr $(if $(findstring ${gdbwaittarget}, ${@}), -XCFA --gdb)  @CONFIG_CFAFLAGS@
 AM_CFLAGS = -g -Wall -Wno-unused-function -fPIC @ARCH_FLAGS@ @CONFIG_CFLAGS@
 AM_CCASFLAGS = -g -Wall -Wno-unused-function @ARCH_FLAGS@ @CONFIG_CFLAGS@

src/main.cc

@@ -17 +17 @@
 #include <execinfo.h>                       // for backtrace, backtrace_symbols
 #include <getopt.h>                         // for no_argument, optind, geto...
-#include <signal.h>                         // for signal, SIGABRT, SIGSEGV
 #include <cassert>                          // for assertf
 #include <cstdio>                           // for fopen, FILE, fclose, stdin
 #include <cstdlib>                          // for exit, free, abort, EXIT_F...
+#include <csignal>                         // for signal, SIGABRT, SIGSEGV
 #include <cstring>                          // for index
 #include <fstream>                          // for ofstream
@@ -96 +96 @@
 DeclarationNode * parseTree = nullptr;                                  // program parse tree
 
+static bool waiting_for_gdb = false; // flag to set cfa-cpp to wait for gdb on start
+
 static std::string PreludeDirector = "";
 
@@ -167 +169 @@
 } // sigAbortHandler
 
-
 int main( int argc, char * argv[] ) {
         FILE * input;                                                                           // use FILE rather than istream because yyin is FILE
@@ -185 +186 @@
         parse_cmdline( argc, argv, filename );                          // process command-line arguments
         CodeGen::FixMain::setReplaceMain( !nomainp );
+
+        if(waiting_for_gdb) {
+                std::cerr << "Waiting for gdb" << std::endl;
+                std::cerr << "run :" << std::endl;
+                std::cerr << "  gdb attach " << getpid() << std::endl;
+                raise(SIGSTOP);
+        }
 
         try {
@@ -198 +206 @@
                 } else {                                                                                // no input file name
                         input = stdin;
+                        std::cerr << "Input from stdin" << std::endl;
                         // if running cfa-cpp directly, might forget to pass -F option. Since this takes from stdin, pass
                         // a fake name along
@@ -438 +447 @@
 
 
-static const char optstring[] = ":hlLmNnpP:S:twW:D:F:";
+static const char optstring[] = ":hlLmNnpP:S:tgwW:D:F:";
 
 enum { PreludeDir = 128 };
@@ -453 +462 @@
         { "statistics", required_argument, nullptr, 'S' },
         { "tree", no_argument, nullptr, 't' },
+        { "gdb", no_argument, nullptr, 'g' },
         { "", no_argument, nullptr, 0 },                                        // -w
         { "", no_argument, nullptr, 0 },                                        // -W
@@ -471 +481 @@
         "<directory> prelude directory for debug/nodebug",      // no flag
         "<option-list> enable profiling information:\n          counters,heap,time,all,none", // -S
-        "build in tree",                                                                        // -t
+        "building cfa standard lib",                                                                    // -t
+        "wait for gdb to attach",                                                                       // -g
         "",                                                                                                     // -w
         "",                                                                                                     // -W
@@ -582 +593 @@
                         Stats::parse_params( optarg );
                         break;
-                  case 't':                                                                             // build in tree
+                  case 't':                                                                             // building cfa stdlib
                         treep = true;
+                        break;
+                  case 'g':                                                                             // wait for gdb
+                        waiting_for_gdb = true;
                         break;
                   case 'w':                                                                             // suppress all warnings, hidden