benlangmuir added a comment.

Did you consider just using a different pragma that triggers this behaviour 
instead of avoiding the crash?  I don't really have a strong preference but I'd 
be interested to hear what you think the pros/cons are.



================
Comment at: lib/Parse/ParseDecl.cpp:5657
+      else
+        PP.getPreprocessorOpts().SeenParserCrashPragma = true;
+    }
----------------
This should return immediately (and keep exiting until ~ParseAST) or it won't 
really behave like a trap.


Repository:
  rC Clang

https://reviews.llvm.org/D41526



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

Reply via email to