https://github.com/ilya-biryukov commented:

I have left quite a few comments, but they are mostly about the code style.
I think this looks really good overall and almost ready to land.

Could you split the change to Sema into a separate PR so that we have time 
profiler changes separately?

https://github.com/llvm/llvm-project/pull/103039
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to