Simplify clang-tidy and explicitly list all relevant rules.
Without using wild-card options, it is easier to switch between
clang-tidy versions without getting new, unexpected findings.
This selective approach also reports less than the previous
configuration which was mostly enabling everything, then only
disabling the rules that were not helpful. Now, we're just
enabling vetted helpful rules.
Output of `xls/dev_tools/run-clang-tidy-cached.sh` is
now a more manageable
```
--- Summary --- (details in xls_clang-tidy.out)
35 [clang-diagnostic-unused-const-variable]
22 [bugprone-argument-comment]
21 [misc-include-cleaner]
20 [clang-diagnostic-error]
15 [modernize-make-unique]
13 [performance-unnecessary-copy-initialization]
9 [clang-diagnostic-unused-function]
7 [google-default-arguments]
6 [abseil-string-find-str-contains]
5 [readability-inconsistent-declaration-parameter-name]
4 [google-readability-function-size]
4 [modernize-use-bool-literals]
3 [bugprone-use-after-move]
3 [modernize-use-equals-default]
3 [readability-container-size-empty]
2 [bugprone-dangling-handle]
2 [clang-diagnostic-thread-safety-analysis]
2 [performance-inefficient-vector-operation]
1 [google-explicit-constructor]
```
PiperOrigin-RevId: 678029545
