mikerice added inline comments.
================ Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:3031 + OrderedEntries[E.getOrder()] = + std::make_tuple(&E, Loc, EntryInfo.ParentName); + ParentFunctions[E.getOrder()] = EntryInfo.ParentName; ---------------- vitalybuka wrote: > mikerice wrote: > > Needed StringRef(EntryInfo.ParentName) for this to compile for me. > StringRef(EntryInfo.ParentName) is ref to the scope-local EntryInfo info, so > you have HWASAN report here Probably meant to change this to const llvm::TargetRegionEntryInfo &EntryInfo to match OffloadTargetRegionEntryInfoActTy? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136601/new/ https://reviews.llvm.org/D136601 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
