aardappel accepted this revision.
aardappel added a comment.
This revision is now accepted and ready to land.

Nice, fairly unintrusive actually. If you desperately wanted to fix the need 
for changing `W` dynamically, you could instead make it allocate a second 
`WasmObjectWriter` to write the `dwo` version? :)



================
Comment at: llvm/lib/MC/MCAsmBackend.cpp:66
+  default:
     report_fatal_error("dwo only supported with ELF");
+  }
----------------
change error message


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85685

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D85685: [WI... Derek Schuff via Phabricator via cfe-commits
    • [PATCH] D85685... Derek Schuff via Phabricator via cfe-commits
    • [PATCH] D85685... Wouter van Oortmerssen via Phabricator via cfe-commits

Reply via email to