ilovepi wrote:

A few comments.

1. I think its better to split the LLVM and clang changes into separate patches.
2. Please update the commit messages w/ the correct [clang] or [llvm] tag.
3. These should probably be some of the last patches to land. Enabling driver 
support in clang + documentation and release notes are normally the last step 
in landing a complex feature. The LLVM flag support is less important about 
where it is in the stack, since it's not exposed externally.

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

Reply via email to