jyknight added a comment.

This patch as it stands is harmless, since as it only defines an internal 
interface, which is unused. So in that sense, it's perfectly fine to commit 
even with the remaining unresolved questions about the correct values to 
return. However, unless we're going to actually use it, adding this code to 
clang is not useful, for the same reason of it only defining an internal 
interface which is unused.

Ultimately, I don't see a reason to commit this, until/unless we are going to 
commit code in Clang using it (which I continue to believe we should not do). 
So I'd say leaving this in a pending state, until a use is going to be 
committed immediately afterwards, seems best.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74918



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

Reply via email to