[PATCH] D78604: Add `//` before the banner displayed in `--print-ir-before/after-all`

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 23 22:46:43 PDT 2020


mehdi_amini added a comment.

In D78604#1999852 <https://reviews.llvm.org/D78604#1999852>, @jpienaar wrote:

> Does it make it valid? Wouldn't you have the same function being redefined?


In TensorFlow: we emit a new file on every invocation right now, and we print a full module. When I get dump for reproduction the only change I make is opening the file and deleting the first line :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78604





More information about the llvm-commits mailing list