[PATCH] D53051: [llvm-tapi] initial commit, supports ELF text stubs

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 30 18:18:53 PDT 2018


echristo added a comment.

Random drive-by bikeshed:

TBEHandlerV1 is probably less than an ideal name for either an implementation or a filename. I'd suggest just calling it TBEHandler and making it possible to version and then subclass later if necessary.

Thoughts?

-eric


Repository:
  rL LLVM

https://reviews.llvm.org/D53051





More information about the llvm-commits mailing list