Tom Stellard <t...@stellard.net> writes: > On Tue, May 13, 2014 at 11:35:36AM +0200, Francisco Jerez wrote: >> Tom Stellard <t...@stellard.net> writes: >> >> > On Tue, May 13, 2014 at 01:23:03AM +0200, Francisco Jerez wrote: >> >> Tom Stellard <thomas.stell...@amd.com> writes: >> >> >> >> > https://bugs.freedesktop.org/show_bug.cgi?id=78581 >> >> > >> >> > CC: "10.1 10.2" <mesa-sta...@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? >> > > There is a FIXME comment higher up in the function that says stderr > should be replaced with an OutputStream parameter once the all the > APIs take OutputStream as a parameter. I can file a bug for this, > but I think for now, we'll have to work around it in clover. > Please do, and add a reference to the Clang bug in the comment too if you don't mind.
Thanks. > -Tom > >> > -Tom >> > >> >> > #if HAVE_LLVM <= 0x0301 >> >> > c.getInvocation().setLangDefaults(clang::IK_OpenCL); >> >> > #else >> >> > -- >> >> > 1.8.1.5 >> > >> > >> > >> > >> >> _______________________________________________ >> >> mesa-dev mailing list >> >> mesa-dev@lists.freedesktop.org >> >> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
pgpNU5oPosHW0.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
