[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