JDevlieghere wrote: I don't think Alex is arguing in favor of keeping the old (wrong) behavior, but the first file looks like this:
``` foundSpec = False if [...] foundSpec = True [...] if foundSpec is False: ``` It's pretty obvious this is a boolean and should use `if not foundSpec`. https://github.com/llvm/llvm-project/pull/91858 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits