https://bugs.llvm.org/show_bug.cgi?id=35959

            Bug ID: 35959
           Summary: print warnings normally in /fallback mode
           Product: clang
           Version: trunk
          Hardware: PC
                OS: Windows NT
            Status: NEW
          Severity: enhancement
          Priority: P
         Component: Driver
          Assignee: unassignedclangb...@nondot.org
          Reporter: tras...@gmail.com
                CC: llvm-bugs@lists.llvm.org

clang prints warning(clang) in fallback mode so MSBuild does not fail the build
because of errors printed by clang. But actually warnings could and should be
printed normally (so MSBuild/VS picks them up) as I still want to benefit from
clang's rich diagnostics even if it fails at some point.

Proof-of-concept:

Index: TextDiagnostic.cpp
===================================================================
--- TextDiagnostic.cpp  (revision 322481)
+++ TextDiagnostic.cpp  (working copy)
@@ -725,7 +725,7 @@
   // makes it more clear whether a message is coming from clang or cl.exe,
   // and it prevents MSBuild from concluding that the build failed just
because
   // there is an "error:" in the output.
-  if (CLFallbackMode)
+  if (CLFallbackMode && Level >= DiagnosticsEngine::Error)
     OS << "(clang)";

   OS << ": ";

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

Reply via email to