llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) <details> <summary>Changes</summary> We don't need to emit the nullptr and compare to it anymore. --- Full diff: https://github.com/llvm/llvm-project/pull/139846.diff 1 Files Affected: - (modified) clang/lib/AST/ByteCode/Compiler.cpp (+2-5) ``````````diff diff --git a/clang/lib/AST/ByteCode/Compiler.cpp b/clang/lib/AST/ByteCode/Compiler.cpp index c7fb5e8466686..96ae1f58ef4a6 100644 --- a/clang/lib/AST/ByteCode/Compiler.cpp +++ b/clang/lib/AST/ByteCode/Compiler.cpp @@ -4107,11 +4107,8 @@ template <class Emitter> bool Compiler<Emitter>::visitBool(const Expr *E) { return true; // Convert pointers to bool. - if (T == PT_Ptr) { - if (!this->emitNull(*T, 0, nullptr, E)) - return false; - return this->emitNE(*T, E); - } + if (T == PT_Ptr) + return this->emitIsNonNullPtr(E); // Or Floats. if (T == PT_Float) `````````` </details> https://github.com/llvm/llvm-project/pull/139846 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
