carlosgalvezp wrote:

I think this patch can be orthogonal to the system headers one, the original 
one from @njames93 contained similar logic I believe (but I chose to not 
include it for simplicity).

It's probably good to apply this change now before modules become mainstream 
and we have debt to deal with, like it happened with my patch :) 

I agree however that there may be other things than Decls to take care of, but 
I don't have a complete picture here. 

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

Reply via email to