[cfe-commits] r166655 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.td include/clang/Basic/TargetInfo.h lib/Basic/Targets.cpp lib/Sema/SemaStmtAsm.cpp test/Misc/warning-flags.c
isanbard at gmail.com
Wed Oct 24 19:06:44 PDT 2012
Yeah. I realize the text of the warning wasn't super. I'm rethinking the whole patch though. As it turns out, people write bad ASM all the time, like
asm("foo %0", "=r" :);
I don't know if it's profitable to warn in this situation or the like.
On Oct 24, 2012, at 6:50 PM, John McCall <rjmccall at apple.com> wrote:
> On Oct 24, 2012, at 5:32 PM, Bill Wendling wrote:
>> Author: void
>> Date: Wed Oct 24 19:32:44 2012
>> New Revision: 166655
>> URL: http://llvm.org/viewvc/llvm-project?rev=166655&view=rev
>> Revert r166647 to rethink the patch...
> FWIW, Eli's review just means that you shouldn't ever add a warning
> without putting it in a warning group. But in general, please don't
> speculatively add warnings that you don't intend to support; the
> diagnostic text here is worryingly wishy-washy.
More information about the cfe-commits