[llvm] [SPIR-V] Cache PartialOrderingVisitor in Splitter instead of rebuilding per call (PR #211198)

Juan Manuel Martinez CaamaƱo via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 22 07:47:45 PDT 2026


================
@@ -313,12 +306,14 @@ class SPIRVStructurizerImpl {
     LoopInfo &LI;
     DomTreeBuilder::BBDomTree DT;
     DomTreeBuilder::BBPostDomTree PDT;
+    std::optional<PartialOrderingVisitor> POV;
----------------
jmmartinez wrote:

If I'm not wrong, `PartialOrderingVisitor` is declared/defined in `SPIRVUtils.h/.cpp`, but only used it by this pass (and both, this pass and the visitor look... scrappy). What do you think about moving it to its own header/source to isolate it?

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


More information about the llvm-commits mailing list