summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMandeep Singh Grang <mgrang@codeaurora.org>2018-04-16 18:56:10 +0000
committerMandeep Singh Grang <mgrang@codeaurora.org>2018-04-16 18:56:10 +0000
commit88a8b269b4f1bb1f2761bef4db028adc9bcaa5fe (patch)
treed6a6b88726780c350e9bb0f65636e82bb801359c
parent70539360e6a743edd257889c00e7856ec3101d0d (diff)
downloadbcm5719-llvm-88a8b269b4f1bb1f2761bef4db028adc9bcaa5fe.tar.gz
bcm5719-llvm-88a8b269b4f1bb1f2761bef4db028adc9bcaa5fe.zip
[RISCV] Fix assert message operator
Summary: Specifying assert message with an || operator makes the compiler interpret it as a bool. Changed it to &&. Reviewers: asb, apazos Reviewed By: asb Subscribers: rbar, johnrusso, simoncook, jordy.potman.lists, sabuasal, niosHD, kito-cheng, shiva0217, zzheng, llvm-commits Differential Revision: https://reviews.llvm.org/D45660 llvm-svn: 330148
-rw-r--r--llvm/lib/Target/RISCV/RISCVISelLowering.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Target/RISCV/RISCVISelLowering.cpp b/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
index 0018a9c750f..1d2a15c319b 100644
--- a/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
+++ b/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
@@ -658,7 +658,7 @@ static bool CC_RISCV(const DataLayout &DL, unsigned ValNo, MVT ValVT, MVT LocVT,
// Handle passing f64 on RV32D with a soft float ABI.
if (XLen == 32 && ValVT == MVT::f64) {
- assert(!ArgFlags.isSplit() && PendingLocs.empty() ||
+ assert(!ArgFlags.isSplit() && PendingLocs.empty() &&
"Can't lower f64 if it is split");
// Depending on available argument GPRS, f64 may be passed in a pair of
// GPRs, split between a GPR and the stack, or passed completely on the
OpenPOWER on IntegriCloud