From e695dc1405e4985b1fca781207068410034a9585 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Sun, 6 Apr 2025 20:09:10 +0200 Subject: [PATCH] LibJS: Don't make Bytecode::Operand::Type an u8 outside of aarch64 This is a slightly mystified attempt to recover the performance regression seen on our JS benchmark runner after 3c2a2bb39f94a and c7bba505eaced. With this change, c7bba505eaced is effectively reverted from the perspective of x86_64. --- Libraries/LibJS/Bytecode/Operand.h | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/Libraries/LibJS/Bytecode/Operand.h b/Libraries/LibJS/Bytecode/Operand.h index 74e2ad369c1..b5052f19012 100644 --- a/Libraries/LibJS/Bytecode/Operand.h +++ b/Libraries/LibJS/Bytecode/Operand.h @@ -13,7 +13,11 @@ namespace JS::Bytecode { class Operand { public: - enum class Type : u8 { + enum class Type +#if ARCH(AARCH64) + : u8 +#endif + { Invalid, Register, Local,