[llvm] improve debug messages in delinearization and dependence analysis (PR #156339)
Sjoerd Meijer via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 2 00:21:39 PDT 2025
================
@@ -354,18 +354,27 @@ void llvm::computeAccessFunctions(ScalarEvolution &SE, const SCEV *Expr,
if (!AR->isAffine())
return;
+ // Clear output vector.
+ Subscripts.clear();
----------------
sjoerdmeijer wrote:
I was going to say that we should add NFC to the subject line, until I noticed this one here.
That looks like it is fixing a bug? I see the same pattern of clearing `Subscripts` and `Sizes` in the other functions below. As a minimum, I think it is worth documenting this API change in the header file and comments?
https://github.com/llvm/llvm-project/pull/156339
More information about the llvm-commits
mailing list