rosefromthedead wrote: @AaronBallman I chose to add it to the existing group because if an #include both uses the wrong case and has backslashes, we'd emit two fixits for the same source span (and I assume that would misbehave). I agree it would be nice to be able to turn it off separately though. I saw that suppressing warnings based on flags is usually handled implicitly inside DiagnosticBuilder which scared me away from e.g. manually checking whether path separator warnings are enabled and then modifying the case insensitivity warning so that only one fixit is emitted for both changes.
Is there a clean way to separate them to be behind different flags? https://github.com/llvm/llvm-project/pull/186770 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
