[PATCH] D86138: [yaml2obj] - Don't crash when `FileHeader` declares an empty `Flags` key in specific situations.
    George Rimar via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Aug 18 06:04:28 PDT 2020
    
    
  
grimar added a comment.
In D86138#2223614 <https://reviews.llvm.org/D86138#2223614>, @jhenderson wrote:
> LGTM. Is there testing for the other flags discussed in that function? If not, we might want to add it.
There is nothing yet, except the `mips-eflags.yaml` test for `obj2yaml` that does it for mips. Though I'd expect to see
a test for `yaml2obj` that uses something different, like `llvm-readobj`. So, yes, we might want to add more testing for all kind of `EF_*`.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86138/new/
https://reviews.llvm.org/D86138
    
    
More information about the llvm-commits
mailing list