[llvm] [llvm][loop-rotate] Allow forcing loop-rotation (PR #82828)

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 26 04:08:44 PST 2024


================
@@ -209,6 +209,15 @@ static cl::opt<bool> EnableLoopFlatten("enable-loop-flatten", cl::init(false),
                                        cl::Hidden,
                                        cl::desc("Enable the LoopFlatten Pass"));
 
+// Experimentally allow loop header duplication. This should allow for better
----------------
fhahn wrote:

FWIW I don't think allowing loop-rotation for all loops to simplify some to memsets is the best way forward here, as I don't see any path towards having this enabled by default.

> If this ends up being profitable, we should drop this flag and
>  // making a code gen option that can be controlled independent of the opt level
> // and exposed through clang.

It is not clear to me what profitable means here?

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


More information about the llvm-commits mailing list