steakhal added a comment.

I'm not against the idea.
However, the implementation seems to be quite complicated, compared to what 
it's actually doing.
Can't we spare the heap allocation somehow? It doesn't seem necessary to me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116924

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

Reply via email to