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