junaire marked 4 inline comments as done. junaire added inline comments.
================ Comment at: clang/include/clang/Interpreter/Interpreter.h:83 /// mangled name. llvm::Expected<llvm::JITTargetAddress> getSymbolAddress(GlobalDecl GD) const; ---------------- sgraenitz wrote: > Most of the Orc and JITLink moved away from `JITTargetAddress` already and > uses `ExecutorAddr` nowadays. I think we should use `ExecutorAddr` from the > beginning in this patch. Thanks, I've created https://reviews.llvm.org/D148434 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141215/new/ https://reviews.llvm.org/D141215 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits