[PATCH] D61767: [llvm-elfabi] Emit ELF header and string table section

Tue Nov 17 00:59:42 PST 2020

jhenderson added inline comments.

================
Comment at: llvm/test/tools/llvm-elfabi/fail-file-write.test:16
+
+# ERR: Permission denied when trying to open
----------------
haowei wrote:
> jhenderson wrote:
> > jhenderson wrote:
> > > Please check the full error here, rather than a partial one. Additionally, I would be suspicious that the error message will be different depending on whether the host OS is Linux or Windows. Please could you check? If you don't have access to both systems, let me know, and I can test the one you can't.
> > Looking at the pre-merge checks this test is failing on Windows as-is.
> I have a windows workstation but it does not have development environment set up for llvm. I will see if I can set it up tomorrow. For now, let me test it on bots first.
Did you get anywhere with resolving this? The test is still failing on the pre-merge bot, as noted near the top of this page.

================
Comment at: llvm/test/tools/llvm-elfabi/output-target-error.test:1
+## Test running llvm-elfabi without specifying correct targets.
+
----------------

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61767/new/

https://reviews.llvm.org/D61767