kparzysz wrote:

> In general, we aim to only depend on libClangDriver and move it to LLVM so 
> that Flang need not depend on Clang or its libraries. If this patch increases 
> the dependency on Clang it might not be acceptable.

Agreed. 👍

@JDPailleux:  It may a good idea to extract the diagnostic engine into a 
separate (from clang) library that could be built on its own.  Still, this 
would require a discussion on Discourse before changing anything.

Flang already handles diagnostic messages on its own, so using clang's 
diagnostic engine it would have to offer clear advantages.  If you want to 
start a conversation, please think of those.

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

Reply via email to