[clang] 875841c - [clang][bytecode] Avoid a getValue() call in builtin_isinf (#152939)
via cfe-commits
cfe-commits at lists.llvm.org
Sun Aug 10 22:11:41 PDT 2025
Author: Timm Baeder
Date: 2025-08-11T07:11:38+02:00
New Revision: 875841c93d20706842df86964568bd9b44436ff6
URL: https://github.com/llvm/llvm-project/commit/875841c93d20706842df86964568bd9b44436ff6
DIFF: https://github.com/llvm/llvm-project/commit/875841c93d20706842df86964568bd9b44436ff6.diff
LOG: [clang][bytecode] Avoid a getValue() call in builtin_isinf (#152939)
Get the APFloat once and work with that, instead of calling isInf() and
potentially isNegative().
Added:
Modified:
clang/lib/AST/ByteCode/InterpBuiltin.cpp
Removed:
################################################################################
diff --git a/clang/lib/AST/ByteCode/InterpBuiltin.cpp b/clang/lib/AST/ByteCode/InterpBuiltin.cpp
index 14f6929037ff5..ad923a52a4ce1 100644
--- a/clang/lib/AST/ByteCode/InterpBuiltin.cpp
+++ b/clang/lib/AST/ByteCode/InterpBuiltin.cpp
@@ -459,12 +459,13 @@ static bool interp__builtin_isinf(InterpState &S, CodePtr OpPC,
const InterpFrame *Frame, bool CheckSign,
const CallExpr *Call) {
const Floating &Arg = S.Stk.pop<Floating>();
- bool IsInf = Arg.isInf();
+ APFloat F = Arg.getAPFloat();
+ bool IsInf = F.isInfinity();
if (CheckSign)
- pushInteger(S, IsInf ? (Arg.isNegative() ? -1 : 1) : 0, Call->getType());
+ pushInteger(S, IsInf ? (F.isNegative() ? -1 : 1) : 0, Call->getType());
else
- pushInteger(S, Arg.isInf(), Call->getType());
+ pushInteger(S, IsInf, Call->getType());
return true;
}
More information about the cfe-commits
mailing list