yonghong-song added inline comments.
================ Comment at: clang/include/clang/AST/Type.h:4768 +private: + std::string BTFTypeTag; + ---------------- aaron.ballman wrote: > I think we should be able to use a `StringRef` instead, rather than have to > worry about allocations. StringRef indeed works as a type. Previously, I used StringRef directly passed by the caller but later the underlying string is somehow freed so I got a "" string for tag. I will revisit this. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111199/new/ https://reviews.llvm.org/D111199 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
