[PATCH] D75278: Use FileDescriptorIsDisplayed in place of direct call to isatty in support library

Douglas Gliner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 9 13:30:51 PDT 2020


dgg5503 added a comment.

@MaskRay thanks for the approval. I just want to make sure you saw my earlier question since I haven't yet updated the description or added a comment:

In D75278#1910426 <https://reviews.llvm.org/D75278#1910426>, @dgg5503 wrote:

> @MaskRay thanks for the feedback. Before I update the summary and code with a comment, I'd like to first confirm that from our discussion about this function, are Windows, Minix, and fully POSIX compliant OS the only hosts supported by LLVM so far?





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

https://reviews.llvm.org/D75278





More information about the llvm-commits mailing list