[PATCH] D116629: Introduce NewPM DOT printers for DomTree

[Daniil Suchkov via Phabricator via llvm-commits]

DaniilSuchkov added inline comments.


================
Comment at: llvm/test/Analysis/Dominators/print-dot-dom.ll:1
+; RUN: opt %s -passes=dot-dom -disable-output
+; RUN: FileCheck %s -input-file=dom.test1.dot -check-prefix=TEST1
----------------
aeubanks wrote:
> ideally the files would go in a temp directory (`%t`) rather than the cwd, but I think the cwd is in the build directory so this is probably fine
> but I think the cwd is in the build directory

Yeah, that's correct. There are other tests that do the same thing, like the test from https://reviews.llvm.org/D90638, and it seems to work fine.


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

https://reviews.llvm.org/D116629