[PATCH] D54939: [llvm-objcopy] Initial COFF support
    Alexander Shaposhnikov via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Dec 13 14:16:54 PST 2018
    
    
  
alexshap added a comment.
sorry about the delay, I've also pinged some other people who know coff better than I do (at the moment).
What do think about keeping the implementation of Reader and Writer in their own files, and (in the future) put into Object.cpp only the logic for
modifying the "intermediate representation" (mutations of COFF) ?
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54939/new/
https://reviews.llvm.org/D54939
    
    
More information about the llvm-commits
mailing list