[PATCH] D88635: [llvm-readobj] Add --coff-tls-directory flag to print TLS Directory & test.
George Rimar via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 8 01:40:40 PDT 2020
grimar accepted this revision.
grimar added a comment.
LGTM with a nit addressed.
================
Comment at: llvm/test/tools/llvm-readobj/COFF/tls-directory.test:159
+# NO-TLS2: TLSDirectory {
+# NO-TLS2-NEXT: }
+
----------------
You don't need to have `NO-TLS2` and `NO-TLS1`. You can just have a single `NO-TLS` and reuse it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D88635/new/
https://reviews.llvm.org/D88635
More information about the llvm-commits
mailing list