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