[Mesa-stable] [Mesa-dev] [PATCH] clover: Prevent Clang from printing number of errors and warnings to stderr.

[Francisco Jerez]

Tom Stellard <tom at stellard.net> writes:

> On Tue, May 13, 2014 at 01:23:03AM +0200, Francisco Jerez wrote:
>> Tom Stellard <thomas.stellard at amd.com> writes:
>> 
>> > https://bugs.freedesktop.org/show_bug.cgi?id=78581
>> >
>> > CC: "10.1 10.2" <mesa-stable at lists.freedesktop.org>
>> > ---
>> >  src/gallium/state_trackers/clover/llvm/invocation.cpp | 4 ++++
>> >  1 file changed, 4 insertions(+)
>> >
>> > diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp b/src/gallium/state_trackers/clover/llvm/invocation.cpp
>> > index 82e262f..c211a06 100644
>> > --- a/src/gallium/state_trackers/clover/llvm/invocation.cpp
>> > +++ b/src/gallium/state_trackers/clover/llvm/invocation.cpp
>> > @@ -187,6 +187,10 @@ namespace {
>> >        c.getLangOpts().NoBuiltin = true;
>> >        c.getTargetOpts().Triple = triple;
>> >        c.getTargetOpts().CPU = processor;
>> > +
>> > +      // This prevents Clang from printing the number of warnings and errors
>> > +      // to stderr.
>> > +      c.getDiagnosticOpts().ShowCarets = false;
>> 
>> Hm...  Why does 'ShowCarets = true' lead Clang to print errors to
>> stderr?
>> 
>
> I'm not really sure.  Here is the relevant code in Clang:
>
> https://github.com/llvm-mirror/clang/blob/master/lib/Frontend/CompilerInstance.cpp#L728
>

It doesn't look like this is the intended behaviour?  Clang bug
probably?

> -Tom
>
>> >  #if HAVE_LLVM <= 0x0301
>> >        c.getInvocation().setLangDefaults(clang::IK_OpenCL);
>> >  #else
>> > -- 
>> > 1.8.1.5
>
>
>
>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 229 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-stable/attachments/20140513/a3b54999/attachment.sig>