[PATCH] D96628: [VPlan] Add plain text (not DOT's digraph) dumps

Andrei Elovikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 14 19:29:51 PDT 2021


a.elovikov added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/vplan-dot-printing.ll:26
+for.body:                                         ; preds = %entry, %for.body
+  %iv = phi i64 [ %iv.next, %for.body ], [ 0, %entry ]
+  %arrayidx = getelementptr inbounds float, float* %y, i64 %iv
----------------
fhahn wrote:
> Nit: it would be good to at least one additional BB in the loop, to make sure the edges between blocks are still printed.
This functionality is being tested in unittests already, so I'm only using this file to target the toggle switch, not the digraph dump by itself.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96628



More information about the llvm-commits mailing list