vabridgers wrote:

As I understand the collective comments so far:

- use `misc`, not `bugprone`. I've heard no affirmations for `bugprone` and one 
suggestion to use `misc`.
- Break up the simulation header files, only include what's actually needed for 
the test
- change the checker name to something like 
`bugprone-nondeterministic-pointer-iteration-order`
- provide testing results, expand upon testing
- I've heard no concrete comments on expanding beyond the few cases this check 
currently addresses. I'll assume that's ok for now.

Next update should address these points. Thanks for the comments!


https://github.com/llvm/llvm-project/pull/110471
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to