hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land.
looks good. ================ Comment at: clang-tools-extra/clangd/refactor/tweaks/DefineOutline.cpp:53 +// template keyword for templated functions. +// FIXME: This is shared with define inline, move them to a common header once +// we have a place for such. ---------------- this function seems trivial, I'd inline them into call sides for both define inline and outline. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69298/new/ https://reviews.llvm.org/D69298 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
