Skip to content

Commit f18dd9e

Browse files
committed
Reapply "[Clang][Interp] __builtin_os_log_format_buffer_size should be an unevaluated builtin (llvm#99895)"
This reverts commit 5f05d5e. Reapply the original commit without the test. The memory leak is caused by a well known problem in the new constant interpreter.
1 parent dd23b34 commit f18dd9e

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

clang/lib/AST/Interp/ByteCodeEmitter.cpp

+2-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@ using namespace clang::interp;
2727
/// Similar information is available via ASTContext::BuiltinInfo,
2828
/// but that is not correct for our use cases.
2929
static bool isUnevaluatedBuiltin(unsigned BuiltinID) {
30-
return BuiltinID == Builtin::BI__builtin_classify_type;
30+
return BuiltinID == Builtin::BI__builtin_classify_type ||
31+
BuiltinID == Builtin::BI__builtin_os_log_format_buffer_size;
3132
}
3233

3334
Function *ByteCodeEmitter::compileFunc(const FunctionDecl *FuncDecl) {

0 commit comments

Comments
 (0)