summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZaara Syeda <syzaara@ca.ibm.com>2021-12-14 16:45:43 +0000
committerZaara Syeda <syzaara@ca.ibm.com>2021-12-14 16:46:37 +0000
commitdd245bab9fbb364faa1581e4f92ba3119a872fba (patch)
treea5ad64ba81554ec8eb106251ae2e614707d6cf0e
parent423f19680a4fc60c106cef102730939c3c7ab7ac (diff)
downloadllvm-dd245bab9fbb364faa1581e4f92ba3119a872fba.tar.gz
[LoopUnroll] Disable loop unroll when user explicitly asks for unroll-and-jam
If a loop isn't forced to be unrolled, we want to avoid unrolling it when there is an explicit unroll-and-jam pragma. This is to prevent automatic unrolling from interfering with the user requested transformation. Differential Revision: https://reviews.llvm.org/D114886
-rw-r--r--llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp25
1 files changed, 25 insertions, 0 deletions
diff --git a/llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp b/llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp
index 39c8b65968aa..893928fb0560 100644
--- a/llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp
+++ b/llvm/lib/Transforms/Scalar/LoopUnrollPass.cpp
@@ -1136,6 +1136,31 @@ static LoopUnrollResult tryToUnrollLoop(
TransformationMode TM = hasUnrollTransformation(L);
if (TM & TM_Disable)
return LoopUnrollResult::Unmodified;
+
+ // If this loop isn't forced to be unrolled, avoid unrolling it when the
+ // parent loop has an explicit unroll-and-jam pragma. This is to prevent
+ // automatic unrolling from interfering with the user requested
+ // transformation.
+ Loop *ParentL = L->getParentLoop();
+ if (ParentL != NULL &&
+ hasUnrollAndJamTransformation(ParentL) == TM_ForcedByUser &&
+ hasUnrollTransformation(L) != TM_ForcedByUser) {
+ LLVM_DEBUG(dbgs() << "Not unrolling loop since parent loop has"
+ << " llvm.loop.unroll_and_jam.\n");
+ return LoopUnrollResult::Unmodified;
+ }
+
+ // If this loop isn't forced to be unrolled, avoid unrolling it when the
+ // loop has an explicit unroll-and-jam pragma. This is to prevent automatic
+ // unrolling from interfering with the user requested transformation.
+ if (hasUnrollAndJamTransformation(L) == TM_ForcedByUser &&
+ hasUnrollTransformation(L) != TM_ForcedByUser) {
+ LLVM_DEBUG(
+ dbgs()
+ << " Not unrolling loop since it has llvm.loop.unroll_and_jam.\n");
+ return LoopUnrollResult::Unmodified;
+ }
+
if (!L->isLoopSimplifyForm()) {
LLVM_DEBUG(
dbgs() << " Not unrolling loop which is not in loop-simplify form.\n");