[clang] [clang][bytecode] Avoid classify()ing logical binary operators (PR #219943)
Timm Baeder via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 31 04:13:56 PDT 2026
https://github.com/tbaederr created https://github.com/llvm/llvm-project/pull/219943
We can just check `isBooleanType()` instead.
>From 2921352cbf37eb9114cc37bc482867f814952ef6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?= <tbaeder at redhat.com>
Date: Mon, 31 Aug 2026 13:10:52 +0200
Subject: [PATCH] one less classify call
---
clang/lib/AST/ByteCode/Compiler.cpp | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/clang/lib/AST/ByteCode/Compiler.cpp b/clang/lib/AST/ByteCode/Compiler.cpp
index a65a0441bba62..55abd51659b46 100644
--- a/clang/lib/AST/ByteCode/Compiler.cpp
+++ b/clang/lib/AST/ByteCode/Compiler.cpp
@@ -1551,7 +1551,6 @@ bool Compiler<Emitter>::VisitLogicalBinOp(const BinaryOperator *E) {
BinaryOperatorKind Op = E->getOpcode();
const Expr *LHS = E->getLHS();
const Expr *RHS = E->getRHS();
- OptPrimType T = classify(E->getType());
if (Op == BO_LOr) {
// Logical OR. Visit LHS and only evaluate RHS if LHS was FALSE.
@@ -1600,9 +1599,10 @@ bool Compiler<Emitter>::VisitLogicalBinOp(const BinaryOperator *E) {
return this->emitPopBool(E);
// For C, cast back to integer type.
- assert(T);
- if (T != PT_Bool)
- return this->emitCast(PT_Bool, *T, E);
+ if (!E->getType()->isBooleanType()) {
+ PrimType T = classifyPrim(E->getType());
+ return this->emitCast(PT_Bool, T, E);
+ }
return true;
}
More information about the cfe-commits
mailing list