[PATCH] D35286: [Dominators] Simplify block and node printing

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 12 16:30:36 PDT 2017


davide added a comment.

In https://reviews.llvm.org/D35286#807394, @kuhar wrote:

> In https://reviews.llvm.org/D35286#807383, @davide wrote:
>
> > This is not your fault but it would be awesome if we introduced a common error handling for the dom, so that we can just call that routine which also calls `flush`.
>
>
> Is there something similar in LLVM that I could use as an example?


it just needs to be a static helper, nothing fancy, just to reduce the amount of `os.flush()` typing :)
see, e.g. r299551 (except that you need to call `os.flush()` and this routine actually returns.


https://reviews.llvm.org/D35286





More information about the llvm-commits mailing list