[clang-tools-extra] Added options to readability-implicit-bool-conversion (PR #120087)

Julian Schmidt via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 19 16:25:07 PST 2024


================

----------------
5chmidti wrote:

You have added `-check-suffix` options to each of your `RUN` lines, but none of your `CHECK-MESSAGES` are actually using it. E.g.,: 
```C++
// CHECK-MESSAGES-TO-BOOL-FALSE
```

Although, instead of using the suffixes to say `FALSE`, you could swap them around to say that the option is enabled, so `-check-suffix:TO-BOOL`. Then, the version with both enabled becomes `-check-suffix:TO-BOOL,FROM-BOOL`.

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


More information about the cfe-commits mailing list