[llvm] [llvm-objcopy][ELF] Disable huge section offset (PR #97036)
    Fangrui Song via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Jul  1 23:32:17 PDT 2024
    
    
  
MaskRay wrote:
> @MaskRay, what do you think about this patch? I've not really got a good understanding of binary output, so can't be 100% confident about any reviewing I do.
Frankly, I hope that we leave llvm-objcopy unchanged for #88878.
Users can request object files with sections exceeding 0x80000000, and llvm-objcopy should handle those requests as intended.
Overly defensive measures often create more problems than they solve. If someone adds a new writer, do we port this output size limiting code to that?
An optional flag for this behavior seems unlikely to be used.
Note: we did introduce a default size limit for yaml2obj (10MiB) https://reviews.llvm.org/D81258 .
The primary reason is that yaml2obj is an internal testing tool, instead of a user-facing one.
https://github.com/llvm/llvm-project/pull/97036
    
    
More information about the llvm-commits
mailing list