[llvm] [clang] [SpecialCaseList] Use glob by default (PR #74809)

David Blaikie via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 8 10:35:42 PST 2023


dwblaikie wrote:

> > Probably would be good to introduce the `-v1` version and require it first, then eventually change the default - so people don't get a silent behavior change? Even the existing users only using `*` and `.` need to change their syntax to migrate to v2, right? They'll need to change `.*` to `*` and `.` to `?` and if they don't, their matches will start behaving strangely without a specific/actionable error message?
> 
> Since `#!special-case-list-v1` is just a comment and `v1` is the default before this change, users can actually add it to the first line today and the behavior won't change until Clang 19.

Right, but they wouldn't know to do that in advance of the behavior change, would they? If they do nothing/don't read any release notes, they'd get a silent behavior change, I think/if I'm understanding this correctly?

https://github.com/llvm/llvm-project/pull/74809


More information about the cfe-commits mailing list