Skip to content

Commit 5f05d5e

Browse files
committed
Revert "[Clang][Interp] __builtin_os_log_format_buffer_size should be an unevaluated builtin (#99895)"
This reverts commit 4572efe. Reason: Introduced a memory leak that broke the sanitizer buildbots. More information available in the original pull request (#99895).
1 parent b5fb7b2 commit 5f05d5e

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

clang/lib/AST/Interp/ByteCodeEmitter.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,7 @@ 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 ||
31-
BuiltinID == Builtin::BI__builtin_os_log_format_buffer_size;
30+
return BuiltinID == Builtin::BI__builtin_classify_type;
3231
}
3332

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

clang/test/CodeGen/builtins.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
// RUN: %clang_cc1 -emit-llvm -o %t %s
22
// RUN: not grep __builtin %t
33
// RUN: %clang_cc1 -emit-llvm -triple x86_64-darwin-apple -o - %s | FileCheck %s
4-
// RUN: %clang_cc1 -emit-llvm -triple x86_64-darwin-apple -fexperimental-new-constant-interpreter -o - %s | FileCheck %s
54

65
int printf(const char *, ...);
76

0 commit comments

Comments
 (0)