[llvm] [CycleInfo] Represent cycles by an opaque handle. NFC (PR #210117)

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 16 10:57:24 PDT 2026


================
@@ -1062,20 +1055,19 @@ const CycleT *getIntDivCycle(const CycleInfoT &CI, const CycleT *Cycle,
 
 template <typename ContextT, typename CycleInfoT, typename CycleT,
           typename BlockT, typename DominatorTreeT>
-const CycleT *
-getOutermostDivergentCycle(const CycleInfoT &CI, const CycleT *Cycle,
-                           const BlockT *DivTermBlock, const BlockT *JoinBlock,
-                           const DominatorTreeT &DT, ContextT &Context) {
+CycleT getOutermostDivergentCycle(const CycleInfoT &CI, CycleT Cycle,
----------------
MaskRay wrote:

The return type `CycleT` refers to `typename CycleT`, which is a GenericCycleRef. The `Cycle -> CycleRef` and de-template will fix the confusion here.

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


More information about the llvm-commits mailing list