[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