[PATCH] D37567: [llvm-readobj] - Teach tool to report error if some section is in multiple COMDAT groups at once.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 16 07:38:01 PDT 2017


>Test stderr and stdout independently instead of using 2>&1.
>
>LGTM with that.
>
>Thanks!
>
>Cheers,
>Rafael

r313459. Thanks for review !

Before commit I had to change error message for LLVM style from
"Error: .foo (3) already in a group .group (2)"
to
"Error: .foo (3) in a group .group1 (4) is already in a group .group (2)"
because first one does not make much sence when printed to stderr along.

George.


More information about the llvm-commits mailing list