[PATCH] D37349: [polly] Fix non-deterministic output due to iteration of unordered ScopArrayInfo
Tobias Grosser via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 31 12:43:36 PDT 2017
grosser added inline comments.
================
Comment at: lib/Transform/MaximalStaticExpansion.cpp:504
+ [](ScopArrayInfo *A, ScopArrayInfo *B) {
+ return A->getName() < B->getName(); });
----------------
Why do you need to sort the arrays. The original data structure is a SetVector and should consequently be deterministic. Should we just change the SmallPtrSet into a SmallVector? Would this be sufficient?
https://reviews.llvm.org/D37349
More information about the llvm-commits
mailing list