[PATCH] D100488: [SystemZ][z/OS] Add IsText Argument to GetFile and GetFileOrSTDIN

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 15 13:18:25 PDT 2021


rnk added a comment.

In D100488#2689494 <https://reviews.llvm.org/D100488#2689494>, @amccarth wrote:

> Personally, I'm not a fan of boolean function parameters because of the inline comments necessary to make the call site understandable.  But it appears to be consistent with LLVM Coding Standards and other APIs, so this looks right to me.

I think it would be a reasonable follow-up change to turn these optional boolean parameters into a flags enum.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100488



More information about the cfe-commits mailing list