[PATCH] D55881: [llvm-objcopy] [COFF] Add support for removing symbols
    Martin Storsjö via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Dec 20 05:49:42 PST 2018
    
    
  
mstorsjo marked 10 inline comments as done.
mstorsjo added a comment.
Marked a few more comments done (for things that I moved to a later patch). The main thing I didn't change much is the Referenced field, which I update with a helper function before it's needed. I could make the Reader call it originally to avoid having to do it in COFFObjcopy.cpp as well, in order to try to keep it up to date at all times.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55881/new/
https://reviews.llvm.org/D55881
    
    
More information about the llvm-commits
mailing list