adamcz added inline comments.

================
Comment at: clang-tools-extra/clangd/quality/CompletionModelCodegen.py:168
+        nline.join(class_members),
+        nline.join(["  float EvaluateTree%d() const;" % tree_num
+                    for tree_num in range(num_trees)]),
----------------
Why are these member functions? Why not keep them in .cc file only, in 
anonymous namespace?

I wonder if that will make compiler inline them and then bring back msan issues 
though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88536

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to