Eugene.Zelenko added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/performance/InefficientArrayTraversalCheck.cpp:63 + hasLHS(ToDecl), + hasRHS(integerLiteral(equals(1)))))), + hasBody(BodyMatcher)) ---------------- njames93 wrote: > Thoughts on this, prefetchers are often able to detect strided access so is > forcing increment to be `1` removing potential functionality. Theoretically index changes may be other than 1. For example, in some electronic simulation each block/device store matrix indexes to contribute (but sparse, not dense, matrix is used). It'll be good idea to look on BLAS and LAPACK (through both are written in Fortran). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116875/new/ https://reviews.llvm.org/D116875 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
