[PATCH] D144009: [obj2yaml] Save offset for segments and size for PHDR
    Fangrui Song via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Feb 14 10:50:02 PST 2023
    
    
  
MaskRay added a comment.
In D144009#4126247 <https://reviews.llvm.org/D144009#4126247>, @treapster wrote:
> Only save offset for the first segment for now to fix tests. But changing tests and saving all offsets is probably better?
Just show what the complete fix is, then update all the tests (usually for obj2yaml there are not many to update).
If you feel that the change may need several iterations and don't want to modify tests back and forth, you can say that.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144009/new/
https://reviews.llvm.org/D144009
    
    
More information about the llvm-commits
mailing list