vbvictor wrote: > Actually, another reason for not putting this in performance is that the > changes made by this check may lead to non-functional changes in the code.
Off-by-default option `StrictMode` may exclude fix-its that involve function with side-effects, etc.. AFAIK, if the check does more than one thing (readability, performance) we should place it under `misc` category, so I weakly in favor of `misc` because it may benefit users in different ways. We shall wait for another opinion. https://github.com/llvm/llvm-project/pull/142324 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
