[polly] [polly] Add profitability check for expanded region. (PR #96548)

Michael Kruse via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 25 01:44:01 PDT 2024


================
@@ -345,6 +345,20 @@ class ScopDetection {
   /// @return True if region is profitable to optimize, false otherwise.
   bool isProfitableRegion(DetectionContext &Context) const;
 
+  /// Check if an expanded region is profitable to optimize.
+  ///
+  /// An expanded region may include basic blocks with memory accesses that
+  /// are not used in loops of the expanded region. These memory accesses add
----------------
Meinersbur wrote:

The actual test looks for any memory access, not just accesses not used in the loop.

https://github.com/llvm/llvm-project/pull/96548


More information about the llvm-commits mailing list