fanju110 wrote:

> If I understand what you are saying correctly, this would involve, for 
> example, duplicating the definition of `enum ProfileInstrKind` in `flang` in 
> this PR, then a second PR that moves it from both `flang` and `clang` to 
> `llvm/Frontend`. This is churn that can be distracting. The changes to 
> `clang` in this case are unlikely to be major and getting it reviewed and 
> approved should not be difficult. I would suggest doing those in this PR.

Thanks for the advice, I agree with you, I'm currently refining the codes

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

Reply via email to