[PATCH] D28094: [ELF] - Implemented support for R_386_PC8/R_386_8 relocations.
    Davide Italiano via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Sat Dec 24 04:12:18 PST 2016
    
    
  
davide added a comment.
I think we can either implement or error out on these relocations, I personally don't care as implementing this relocations is literally 3 lines of code, but wait for a second opinion.
What I'm most worried about is that we should never produce a broken output given valid input, we should emit a diagnostic and exit.
https://reviews.llvm.org/D28094
    
    
More information about the llvm-commits
mailing list