[PATCH] D71025: [LoopFusion] Restrict loop fusion to rotated loops.

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 5 08:55:46 PST 2019


Meinersbur accepted this revision.
Meinersbur added a comment.
This revision is now accepted and ready to land.

[suggestion] Add a test case to check for "not rotated" to be rejected.



================
Comment at: llvm/test/Transforms/LoopFusion/cannot_fuse.ll:270
 ; CHECK-NEXT: [[LOOP2PREHEADER]]
-; CHECK: Memory dependencies do not allow fusion!
+; CHECK: Fusion candidate does not have empty preheader. Not fusing.
 ; CHECK: Loop Fusion complete
----------------
Does this test something different now?


================
Comment at: llvm/test/Transforms/LoopFusion/diagnostics_missed.ll:3-4
+
+; ModuleID = '<stdin>'
+source_filename = "<stdin>"
 target datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128"
----------------
[nit] can be removed


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71025/new/

https://reviews.llvm.org/D71025





More information about the llvm-commits mailing list