srishti-pm added a comment.

> Thanks for improving the doc! Are you moving this to be used in 
> canonicalization next?
> I think a first good step would be to make it a pattern and test it with a 
> pass that applies it in the greedy rewriter. I would land this first and then 
> try to enable this in the canonicalized.

Thanks! I hope it is unambiguous and clear now! Yes, I will do the "moving this 
to be used in canonicalization" now. Thanks for the suggestion. The plan sounds 
good.

> Also, have you thought already about how to get rid of string manipulation?

No, I haven't. I'm still thinking about this. Meanwhile, I'll also address the 
other comments given here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124750

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

Reply via email to