[clang] a4ce870 - [clang][Interp][NFC] Add missing comment

Timm Bäder via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 16 10:20:40 PST 2024


Author: Timm Bäder
Date: 2024-02-16T19:20:28+01:00
New Revision: a4ce870859a2d8b5ce8b92732594089e2a81b4fb

URL: https://github.com/llvm/llvm-project/commit/a4ce870859a2d8b5ce8b92732594089e2a81b4fb
DIFF: https://github.com/llvm/llvm-project/commit/a4ce870859a2d8b5ce8b92732594089e2a81b4fb.diff

LOG: [clang][Interp][NFC] Add missing comment

Looks like I lost this last time.

Added: 
    

Modified: 
    clang/lib/AST/Interp/ByteCodeEmitter.cpp

Removed: 
    


################################################################################
diff  --git a/clang/lib/AST/Interp/ByteCodeEmitter.cpp b/clang/lib/AST/Interp/ByteCodeEmitter.cpp
index 2b9ff96daa6c1d..60d8afecb2b3bb 100644
--- a/clang/lib/AST/Interp/ByteCodeEmitter.cpp
+++ b/clang/lib/AST/Interp/ByteCodeEmitter.cpp
@@ -22,6 +22,11 @@
 using namespace clang;
 using namespace clang::interp;
 
+/// Unevaluated builtins don't get their arguments put on the stack
+/// automatically. They instead operate on the AST of their Call
+/// Expression.
+/// Similar information is available via ASTContext::BuiltinInfo,
+/// but that is not correct for our use cases.
 static bool isUnevaluatedBuiltin(unsigned BuiltinID) {
   return BuiltinID == Builtin::BI__builtin_classify_type;
 }


        


More information about the cfe-commits mailing list