diff options
author | Richard Trieu <rtrieu@google.com> | 2016-02-18 22:09:30 +0000 |
---|---|---|
committer | Richard Trieu <rtrieu@google.com> | 2016-02-18 22:09:30 +0000 |
commit | 7a08381403b54cd8998f3c922f18b65867e3c07c (patch) | |
tree | 888b16f3115d38f20de5fefd2d736c173a4fdc76 /llvm/lib/IR/Function.cpp | |
parent | ec3d33274887bc59ff3bfdba90494a63850680cc (diff) | |
download | llvm-7a08381403b54cd8998f3c922f18b65867e3c07c.zip llvm-7a08381403b54cd8998f3c922f18b65867e3c07c.tar.gz llvm-7a08381403b54cd8998f3c922f18b65867e3c07c.tar.bz2 |
Remove uses of builtin comma operator.
Cleanup for upcoming Clang warning -Wcomma. No functionality change intended.
llvm-svn: 261270
Diffstat (limited to 'llvm/lib/IR/Function.cpp')
-rw-r--r-- | llvm/lib/IR/Function.cpp | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/llvm/lib/IR/Function.cpp b/llvm/lib/IR/Function.cpp index da1dded..a0eec5f 100644 --- a/llvm/lib/IR/Function.cpp +++ b/llvm/lib/IR/Function.cpp @@ -882,11 +882,17 @@ bool Function::hasAddressTaken(const User* *PutOffender) const { const User *FU = U.getUser(); if (isa<BlockAddress>(FU)) continue; - if (!isa<CallInst>(FU) && !isa<InvokeInst>(FU)) - return PutOffender ? (*PutOffender = FU, true) : true; + if (!isa<CallInst>(FU) && !isa<InvokeInst>(FU)) { + if (PutOffender) + *PutOffender = FU; + return true; + } ImmutableCallSite CS(cast<Instruction>(FU)); - if (!CS.isCallee(&U)) - return PutOffender ? (*PutOffender = FU, true) : true; + if (!CS.isCallee(&U)) { + if (PutOffender) + *PutOffender = FU; + return true; + } } return false; } |