[PATCH] D68693: [Tests] Output of od can be lower or upper case (llvm-objcopy/yaml2obj).

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 30 14:00:29 PDT 2020


MaskRay added a comment.

I know I am late, but when you re-committed, you should have updated this sentence to mention a concrete example other than a fuzzy mention about the existence:

> However, there are also platforms which use uppercase letter.

Probably https://reviews.llvm.org/D68146#1689253 "z/OS USS". Ideally, include a dump of `od` on that system.

The uppercase problem is a pretty unfortunate fact and `--ignore-case` is mostly hidden knowledge which can only be learned by informed by the people caring about the platform.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68693





More information about the llvm-commits mailing list