[llvm] [Support][NFC] Use single predecessor array in DomTreeConstr (PR #207535)

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 5 07:17:27 PDT 2026


================
@@ -314,8 +319,12 @@ template <typename DomTreeT> struct SemiNCAInfo {
 
       // Initialize the semi dominator to point to the parent node.
       WInfo.Semi = WInfo.Parent;
-      for (unsigned N : WInfo.ReverseChildren) {
-        unsigned SemiU = NumToInfo[eval(N, i + 1, EvalStack, NumToInfo)]->Semi;
+      unsigned RCIdx = WInfo.ReverseChildrenStart;
----------------
MaskRay wrote:

Well, I disagree with this. It's more important to drop the confusing sentinel element. List appending actually saves one `-1`.

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


More information about the llvm-commits mailing list