[PATCH] D66149: [BinaryFormat] Teach identify_magic about Tapi files.
    Cyndy Ishida via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Aug 13 16:29:00 PDT 2019
    
    
  
cishida marked an inline comment as done.
cishida added inline comments.
================
Comment at: llvm/include/llvm/BinaryFormat/Magic.h:52
     pdb,                 ///< Windows PDB debug info file
+    tapi_file,           ///< Text-based Dynamic Library Stub file
   };
----------------
steven_wu wrote:
> tapi_file is not very clear what it actually refer to. Maybe something like tapi_dylib_stub (I am not very happy with this as well)?
what about tbd_file? or tapi_tbd_file? that way it can be deduced that tbd is short for the comment provided and would match the file extension?
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66149/new/
https://reviews.llvm.org/D66149
    
    
More information about the llvm-commits
mailing list