Szelethus added a comment.

In D75682#1940901 <https://reviews.llvm.org/D75682#1940901>, @balazske wrote:

> So what is missing or wrong in this change to be accepted?


This change is functional but is not testable. But the high level idea is 
great! I think we should maybe merge this with the followup, as you originally 
intended to.

Also, judging from the fact that how differently we imagined this to go (even 
though imo your idea is superior to mine), we should explain what our state 
splitting ideology is in the code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75682



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

Reply via email to