[PATCH] D61674: [llvm-objcopy] Improve error message for unrecognised archive member

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 8 06:41:50 PDT 2019


grimar added inline comments.


================
Comment at: test/tools/llvm-objcopy/ELF/archive-unknown-members.test:31
+## Verify that the first member was not modified, if a later member could not
+## be recognised.
+# RUN: cmp %t.o %t1.o
----------------
MaskRay wrote:
> grimar wrote:
> > recognised->recognized :]
> recognised is not wrong..
As far I understand there is a difference between British and American English.
Since our error messages spelling "recognized", i.e. in American way, I think it is good for
comments to be consistent. See rL360252


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61674





More information about the llvm-commits mailing list