[PATCH] D153695: [clang][Interp] Fix passing parameters of composite type

[Shafik Yaghmour via Phabricator via cfe-commits]

shafik added inline comments.


================
Comment at: clang/lib/AST/Interp/ByteCodeEmitter.h:71
 
-  /// Parameter indices.
-  llvm::DenseMap<const ParmVarDecl *, unsigned> Params;
+  /// Parameter indices. <AstNode*, pair<Offset, IsPrimitive>>
+  llvm::DenseMap<const ParmVarDecl *, ParamOffset> Params;
----------------
I don't understand the additional comment,


================
Comment at: clang/lib/AST/Interp/Context.cpp:30
 bool Context::isPotentialConstantExpr(State &Parent, const FunctionDecl *FD) {
+  llvm::errs() << __PRETTY_FUNCTION__ << "\n";
+  FD->dump();
----------------
left over debugging code?


================
Comment at: clang/lib/AST/Interp/Context.cpp:56
 bool Context::evaluateAsRValue(State &Parent, const Expr *E, APValue &Result) {
+  llvm::errs() << __PRETTY_FUNCTION__ << "\n";
+  E->dump();
----------------
Left over debugging code?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D153695/new/

https://reviews.llvm.org/D153695