Author: Timm Bäder Date: 2023-06-27T11:36:42+02:00 New Revision: c3196947312e530960fde1b8849a4a848a5095f3
URL: https://github.com/llvm/llvm-project/commit/c3196947312e530960fde1b8849a4a848a5095f3 DIFF: https://github.com/llvm/llvm-project/commit/c3196947312e530960fde1b8849a4a848a5095f3.diff LOG: [clang][Interp][NFC] Compare std::optional variables directly We can do that and we already checked that they aren't nullopt before. Added: Modified: clang/lib/AST/Interp/ByteCodeExprGen.cpp Removed: ################################################################################ diff --git a/clang/lib/AST/Interp/ByteCodeExprGen.cpp b/clang/lib/AST/Interp/ByteCodeExprGen.cpp index 8558ba0ba1ee7..86200f9ad0831 100644 --- a/clang/lib/AST/Interp/ByteCodeExprGen.cpp +++ b/clang/lib/AST/Interp/ByteCodeExprGen.cpp @@ -230,7 +230,7 @@ bool ByteCodeExprGen<Emitter>::VisitBinaryOperator(const BinaryOperator *BO) { // Pointer arithmetic special case. if (BO->getOpcode() == BO_Add || BO->getOpcode() == BO_Sub) { - if (*T == PT_Ptr || (*LT == PT_Ptr && *RT == PT_Ptr)) + if (T == PT_Ptr || (LT == PT_Ptr && RT == PT_Ptr)) return this->VisitPointerArithBinOp(BO); } _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits