[PATCH] D87786: [LoopUnrollAndJam] Allow unroll and jam loops forced by user.

[Whitney Tsang via Phabricator via llvm-commits]

Whitney added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopUnrollAndJamPass.cpp:303-304
   // Exit early if unrolling is disabled.
   if (!UP.UnrollAndJam || UP.UnrollAndJamInnerLoopThreshold == 0)
     return LoopUnrollResult::Unmodified;
 
----------------
Meinersbur wrote:
> Seems the culprit was probably this line, making the backend's decision to not unroll a higher priority than the user's.
I think now we get the priority right, 1. Option `AllowUnrollAndJam` 2. User pragma 3. backend's decision. :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87786