[llvm] [LV] Remove DataAndControlFlowWithoutRuntimeCheck. NFC (PR #183762)

David Sherwood via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 2 02:04:29 PST 2026


================
@@ -1327,11 +1324,10 @@ class LoopVectorizationCostModel {
   }
 
   /// Returns the TailFoldingStyle that is best for the current loop.
-  TailFoldingStyle getTailFoldingStyle(bool IVUpdateMayOverflow = true) const {
+  TailFoldingStyle getTailFoldingStyle() const {
     if (!ChosenTailFoldingStyle)
       return TailFoldingStyle::None;
-    return IVUpdateMayOverflow ? ChosenTailFoldingStyle->first
-                               : ChosenTailFoldingStyle->second;
+    return *ChosenTailFoldingStyle;
----------------
david-arm wrote:

nit: Can this just be a one-liner, i.e.

```
  return ChosenTailFoldingStyle ? *ChosenTailFoldingStyle : TailFoldingStyle::None;
```

In fact, do we still need `ChosenTailFoldingStyle` to be optional?

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


More information about the llvm-commits mailing list