[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