dblaikie added a comment.

Generally looks OK, but could you explain more about the problems 
with/motivation for the declaration/definition issue? (probably worth noting in 
the patch description maybe comment in code and test)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100567

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

Reply via email to