[llvm] r178144 - Silence warning about mixing || in &&, fix up 80-cols.

David Blaikie dblaikie at gmail.com
Wed Mar 27 16:35:07 PDT 2013


On Wed, Mar 27, 2013 at 8:03 AM, Benjamin Kramer
<benny.kra at googlemail.com> wrote:
> Author: d0k
> Date: Wed Mar 27 10:03:14 2013
> New Revision: 178144
>
> URL: http://llvm.org/viewvc/llvm-project?rev=178144&view=rev
> Log:
> Silence warning about mixing || in &&, fix up 80-cols.

Thanks for the cleanup - not sure why I didn't see the ||/&& warning, though...

>
> Modified:
>     llvm/trunk/lib/IR/DebugInfo.cpp
>
> Modified: llvm/trunk/lib/IR/DebugInfo.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/IR/DebugInfo.cpp?rev=178144&r1=178143&r2=178144&view=diff
> ==============================================================================
> --- llvm/trunk/lib/IR/DebugInfo.cpp (original)
> +++ llvm/trunk/lib/IR/DebugInfo.cpp Wed Mar 27 10:03:14 2013
> @@ -631,7 +631,9 @@ MDNode *DIDerivedType::getObjCProperty()
>
>  /// \brief Set the array of member DITypes.
>  void DICompositeType::setTypeArray(DIArray Elements, DIArray TParams) {
> -  assert(!TParams || DbgNode->getNumOperands() == 15 && "If you're setting the template parameters this should include a slot for that");
> +  assert((!TParams || DbgNode->getNumOperands() == 15) &&
> +         "If you're setting the template parameters this should include a slot "
> +         "for that!");
>    TrackingVH<MDNode> N(*this);
>    N->replaceOperandWith(11, Elements);
>    if (TParams)
>
>
> _______________________________________________
> 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