[all-commits] [llvm/llvm-project] f51427: [Polly][Unroll] Fix unroll_double test.

Michael Kruse via All-commits all-commits at lists.llvm.org
Tue Mar 16 07:01:50 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f51427afb5333e5dd2eb04ea4630037667c64553
      https://github.com/llvm/llvm-project/commit/f51427afb5333e5dd2eb04ea4630037667c64553
  Author: Michael Kruse <llvm-project at meinersbur.de>
  Date:   2021-03-16 (Tue, 16 Mar 2021)

  Changed paths:
    M polly/lib/Transform/ScheduleTreeTransform.cpp
    M polly/test/ScheduleOptimizer/ManualOptimization/unroll_double.ll

  Log Message:
  -----------
  [Polly][Unroll] Fix unroll_double test.

We enumerated the cross product Domain x Scatter, but sorted only be the
scatter key. In case there are are multiple statement instances per
scatter value, the order between statement instances of the same loop
iteration was undefined.

Propertly enumerate and sort only by the scatter value, and group the
domains using the scatter dimension again.

Thanks to Leonard Chan for the report.




More information about the All-commits mailing list