On Dec 16, 2007, at 1:21 PM, Chris Lattner wrote: > > On Dec 14, 2007, at 5:30 PM, Dale Johannesen wrote: > >> Author: johannes >> Date: Fri Dec 14 19:30:32 2007 >> New Revision: 45054 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=45054&view=rev >> Log: >> Suppress a valid warning for llvm. > > Having instcombine emit this warning seems like a really bad thing to > do. Should we just stop the optimizer from emitting warnings like > this? > > -Chris
It's a useful thing to tell users about (users, unlike testsuites and benchmarks, are generally willing to change obviously wrong source). Will it get detected anywhere else, particularly in the IPO case? >> Modified: >> llvm-gcc-4.2/trunk/gcc/testsuite/gcc.c-torture/compile/930411-1.c >> >> Modified: llvm-gcc-4.2/trunk/gcc/testsuite/gcc.c-torture/compile/ >> 930411-1.c >> URL: http://llvm.org/viewvc/llvm-project/llvm-gcc-4.2/trunk/gcc/ >> testsuite/gcc.c-torture/compile/930411-1.c? >> rev=45054&r1=45053&r2=45054&view=diff >> >> = >> = >> = >> = >> = >> = >> = >> = >> ===================================================================== >> = >> --- llvm-gcc-4.2/trunk/gcc/testsuite/gcc.c-torture/compile/ >> 930411-1.c (original) >> +++ llvm-gcc-4.2/trunk/gcc/testsuite/gcc.c-torture/compile/ >> 930411-1.c Fri Dec 14 19:30:32 2007 >> @@ -34,3 +34,5 @@ >> i1 = 1; >> goto L3; >> } >> +/* LLVM LOCAL caller-callee mismatch message can't be suppressed */ >> +/* { dg-prune-output "arguments were dropped" } */ >> >> >> _______________________________________________ >> llvm-commits mailing list >> llvm-commits@cs.uiuc.edu >> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits > > _______________________________________________ > llvm-commits mailing list > llvm-commits@cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits _______________________________________________ llvm-commits mailing list llvm-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
