[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