[llvm-commits] [llvm] r171119 - /llvm/trunk/lib/Transforms/InstCombine/InstCombineCalls.cpp
Bob Wilson
bob.wilson at apple.com
Wed Dec 26 20:03:56 PST 2012
Thanks, Nick. This code was the main thing that prompted my change to add emitWarning to LLVMContext. I think I'll just revert that when I get a chance.
On Dec 26, 2012, at 2:00 PM, Nick Lewycky <nicholas at mxc.ca> wrote:
> Author: nicholas
> Date: Wed Dec 26 16:00:35 2012
> New Revision: 171119
>
> URL: http://llvm.org/viewvc/llvm-project?rev=171119&view=rev
> Log:
> Remove mid-optimizer warning. This situation should be handled differently,
> such as by a compiler warning, a check in clang -fsanitizer=undefined, being
> optimized to unreachable, or a combination of the above. PR14722.
>
> Modified:
> llvm/trunk/lib/Transforms/InstCombine/InstCombineCalls.cpp
>
> Modified: llvm/trunk/lib/Transforms/InstCombine/InstCombineCalls.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/InstCombine/InstCombineCalls.cpp?rev=171119&r1=171118&r2=171119&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Transforms/InstCombine/InstCombineCalls.cpp (original)
> +++ llvm/trunk/lib/Transforms/InstCombine/InstCombineCalls.cpp Wed Dec 26 16:00:35 2012
> @@ -1153,11 +1153,8 @@
>
> // If we are removing arguments to the function, emit an obnoxious warning.
> if (FT->getNumParams() < NumActualArgs) {
> - if (!FT->isVarArg()) {
> - FT->getContext().emitWarning("while resolving call to function '" +
> - Callee->getName() +
> - "' arguments were dropped");
> - } else {
> + // TODO: if (!FT->isVarArg()) this call may be unreachable. PR14722
> + if (FT->isVarArg()) {
> // Add all of the arguments in their promoted form to the arg list.
> for (unsigned i = FT->getNumParams(); i != NumActualArgs; ++i, ++AI) {
> Type *PTy = getPromotedType((*AI)->getType());
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
More information about the llvm-commits
mailing list