[clang] [clang][bytecode] Remove unused local variable (PR #207480)
Timm Baeder via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 3 21:23:36 PDT 2026
https://github.com/tbaederr created https://github.com/llvm/llvm-project/pull/207480
It's unnecessary and broke a builder: https://lab.llvm.org/buildbot/#/builders/228/builds/3735
>From b21ac923c38b0aa81424f0522fe5c329a69c92fc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?= <tbaeder at redhat.com>
Date: Sat, 4 Jul 2026 06:22:47 +0200
Subject: [PATCH] [clang][bytecode] Remove unused local variable
---
clang/lib/AST/ByteCode/Interp.cpp | 7 -------
1 file changed, 7 deletions(-)
diff --git a/clang/lib/AST/ByteCode/Interp.cpp b/clang/lib/AST/ByteCode/Interp.cpp
index dc93cc4fd698c..b099a3e8571f7 100644
--- a/clang/lib/AST/ByteCode/Interp.cpp
+++ b/clang/lib/AST/ByteCode/Interp.cpp
@@ -3231,13 +3231,6 @@ PRESERVE_NONE static bool BCP(InterpState &S, CodePtr OpPC, int32_t Offset,
// We have already evaluated this speculation's EndSpeculation opcode.
assert(S.SpeculationDepth == DepthBefore - 1);
- // Jump to end label. This is a little tricker than just RealPC += Offset
- // because our usual jump instructions don't have any arguments, to the offset
- // we get is a little too much and we need to subtract the size of the
- // bool and PrimType arguments again.
- int32_t ParamSize = align(sizeof(PrimType));
- assert(Offset >= ParamSize);
-
return true;
}
More information about the cfe-commits
mailing list