[PATCH] D97785: [SystemZ][z/OS] Distinguish between text and binary files on z/OS

Abhina Sree via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 22 06:53:08 PDT 2021


abhina.sreeskantharajan added a comment.

In D97785#2638824 <https://reviews.llvm.org/D97785#2638824>, @MaskRay wrote:

> The recommended named parameter form is probably `/*Param=*/something`. It is liked by both https://clang.llvm.org/extra/clang-tidy/checks/bugprone-argument-comment.html and clang-format.
> Another form look to me as well, if both the two tools like it.

Thanks, I've created the following patch https://reviews.llvm.org/D99072 to fix the parameter formatting.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97785



More information about the cfe-commits mailing list