diff options
| author | David L. Jones <dlj@google.com> | 2017-01-13 21:02:41 +0000 |
|---|---|---|
| committer | David L. Jones <dlj@google.com> | 2017-01-13 21:02:41 +0000 |
| commit | 41cecba8e987107c5d8ca4cfb0affde7b4d371a2 (patch) | |
| tree | cbb86819596bee7ac026d68d08a97482884f3def /llvm/lib/Transforms/Utils/ValueMapper.cpp | |
| parent | 64dc9be7b48e3e3ec1f7dec270bf433d2084915a (diff) | |
| download | bcm5719-llvm-41cecba8e987107c5d8ca4cfb0affde7b4d371a2.tar.gz bcm5719-llvm-41cecba8e987107c5d8ca4cfb0affde7b4d371a2.zip | |
"Use" lambda captures which are otherwise only used in asserts. NFC
Summary:
The LLVM coding standards recommend "using" values that are only
needed by asserts:
http://llvm.org/docs/CodingStandards.html#assert-liberally
Without this change, LLVM cannot bootstrap with -Werror as the second
stage fails with this new warning:
https://reviews.llvm.org/rL291905
See also the previous fixes:
https://reviews.llvm.org/rL291916
https://reviews.llvm.org/rL291939
https://reviews.llvm.org/rL291940
https://reviews.llvm.org/rL291941
Reviewers: rsmith
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D28695
llvm-svn: 291957
Diffstat (limited to 'llvm/lib/Transforms/Utils/ValueMapper.cpp')
| -rw-r--r-- | llvm/lib/Transforms/Utils/ValueMapper.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/llvm/lib/Transforms/Utils/ValueMapper.cpp b/llvm/lib/Transforms/Utils/ValueMapper.cpp index 0e9baaf8649..f77c10b6dd4 100644 --- a/llvm/lib/Transforms/Utils/ValueMapper.cpp +++ b/llvm/lib/Transforms/Utils/ValueMapper.cpp @@ -681,6 +681,7 @@ void MDNodeMapper::mapNodesInPOT(UniquedGraph &G) { remapOperands(*ClonedN, [this, &D, &G](Metadata *Old) { if (Optional<Metadata *> MappedOp = getMappedOp(Old)) return *MappedOp; + (void)D; assert(G.Info[Old].ID > D.ID && "Expected a forward reference"); return &G.getFwdReference(*cast<MDNode>(Old)); }); |
