efriedma added inline comments.

================
Comment at: lib/Frontend/PrintPreprocessedOutput.cpp:566
+  MoveToLine(Loc);
+  OS << "#pragma " << "clang assume_nonnull end";
+  setEmittedDirectiveOnThisLine();
----------------
Extra "<<"?


================
Comment at: test/Preprocessor/pragma_assume_nonnull.c:29
+   return bar(0) + foo(0); // expected-warning {{null passed to a callee that 
requires a non-null argument}}
+#endif
+}
----------------
This testcase is a little more complicated than it needs to be (the 
"#if"/"#ifndef" doesn't really add anything).


https://reviews.llvm.org/D37861



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

Reply via email to