[PATCH] D69456: [globalisel][docs] Rewrite the pipeline overview

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 29 12:00:34 PDT 2019


dsanders marked 2 inline comments as done.
dsanders added a comment.

Thanks. llvm.org doesn't seem to be have the previous changes yet so I've asked on llvm-dev about that. Maybe the update mechanism is broken



================
Comment at: llvm/docs/GlobalISel/Pipeline.rst:78
+
+Of course, combiners can be inserted in other places too. Also passes can be
+replaced entirely so long as their task is complete as shown in this (more
----------------
volkan wrote:
> It would be nice to describe combiner too like the other passes.
I added a brief mention. I'll expand on it when I get to that pass


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69456/new/

https://reviews.llvm.org/D69456





More information about the llvm-commits mailing list