[llvm] r310022 - Prevent unused warning in non-assert builds (introduced in r310014).
Adrian Prantl via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 3 23:00:19 PDT 2017
Thanks!
-- adrian
> On Aug 3, 2017, at 10:05 PM, Daniel Jasper via llvm-commits <llvm-commits at lists.llvm.org> wrote:
>
> Author: djasper
> Date: Thu Aug 3 22:05:29 2017
> New Revision: 310022
>
> URL: http://llvm.org/viewvc/llvm-project?rev=310022&view=rev
> Log:
> Prevent unused warning in non-assert builds (introduced in r310014).
>
> Modified:
> llvm/trunk/lib/IR/DIBuilder.cpp
>
> Modified: llvm/trunk/lib/IR/DIBuilder.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/IR/DIBuilder.cpp?rev=310022&r1=310021&r2=310022&view=diff
> ==============================================================================
> --- llvm/trunk/lib/IR/DIBuilder.cpp (original)
> +++ llvm/trunk/lib/IR/DIBuilder.cpp Thu Aug 3 22:05:29 2017
> @@ -678,8 +678,8 @@ DIExpression *DIBuilder::createFragmentE
> if (Op.getOp() == dwarf::DW_OP_LLVM_fragment) {
> // Make the new offset point into the existing fragment.
> uint64_t FragmentOffsetInBits = Op.getArg(0);
> - uint64_t FragmentSizeInBits = Op.getArg(1);
> - assert((OffsetInBits + SizeInBits <= FragmentSizeInBits) &&
> + // Op.getArg(1) is FragmentSizeInBits.
> + assert((OffsetInBits + SizeInBits <= Op.getArg(1)) &&
> "new fragment outside of original fragment");
> OffsetInBits += FragmentOffsetInBits;
> break;
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
More information about the llvm-commits
mailing list