[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