cdevadas wrote: I observed something while porting this pass. The analysis LiveIntervals (LIS) uses the SlotIndexes (SI). There is no explicit use of SI in this pass. If we have to preserve LIS, it required us to preserve SI as well. When I initially failed to preserve SI, the following lit test went into an infinite loop (the cleanup phase hung). llvm/test/CodeGen/AMDGPU/lower-control-flow-live-intervals.mir. Then I realized that the Legacy pass flow preserved SI in getAnalysisUsage(). Is it expected to preserve all the dependent analyses alongside a parent analysis?
https://github.com/llvm/llvm-project/pull/123045 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
