[clang] 79eed76 - [clang][bytecode][NFC] Remove incorrect comment (#139571)
via cfe-commits
cfe-commits at lists.llvm.org
Mon May 12 23:09:29 PDT 2025
Author: Timm Baeder
Date: 2025-05-13T08:09:26+02:00
New Revision: 79eed76c58caedd9dc7c68b38c0dbc08d3ff3da3
URL: https://github.com/llvm/llvm-project/commit/79eed76c58caedd9dc7c68b38c0dbc08d3ff3da3
DIFF: https://github.com/llvm/llvm-project/commit/79eed76c58caedd9dc7c68b38c0dbc08d3ff3da3.diff
LOG: [clang][bytecode][NFC] Remove incorrect comment (#139571)
We don't create function frames for builtin functions anymore.
Added:
Modified:
clang/lib/AST/ByteCode/InterpFrame.cpp
Removed:
################################################################################
diff --git a/clang/lib/AST/ByteCode/InterpFrame.cpp b/clang/lib/AST/ByteCode/InterpFrame.cpp
index e4bd4a6ba7656..a849ac71a7263 100644
--- a/clang/lib/AST/ByteCode/InterpFrame.cpp
+++ b/clang/lib/AST/ByteCode/InterpFrame.cpp
@@ -133,11 +133,7 @@ static bool shouldSkipInBacktrace(const Function *F) {
}
void InterpFrame::describe(llvm::raw_ostream &OS) const {
- // We create frames for builtin functions as well, but we can't reliably
- // diagnose them. The 'in call to' diagnostics for them add no value to the
- // user _and_ it doesn't generally work since the argument types don't always
- // match the function prototype. Just ignore them.
- // Similarly, for lambda static invokers, we would just print __invoke().
+ // For lambda static invokers, we would just print __invoke().
if (const auto *F = getFunction(); F && shouldSkipInBacktrace(F))
return;
More information about the cfe-commits
mailing list