[PATCH] D81156: [Support] Add stream tie function and use it for errs()

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 9 02:42:49 PDT 2020


jhenderson added inline comments.


================
Comment at: llvm/include/llvm/Support/raw_ostream.h:307
+  /// stream. Specifying a nullptr unties the stream.
+  void tie(raw_ostream *TieTo) { TiedStream = TieTo; }
+
----------------
MaskRay wrote:
> `TieTo` -> `TiedTo`
I'm not going to make this change, since this function tells the class to tie itself to `TieTo`. `TiedTo` would imply it has already been tied to it, which would be false.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81156





More information about the llvm-commits mailing list