[PATCH] D127165: [DirectX][MC] Add MC support for DXContainer

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 9 08:43:55 PDT 2022


rengolin added a comment.

I don't know much about DX but this looks like object emission to me. :)

I guess any problem from these changes will only show up in DX lowering, so not very high risk to other targets...



================
Comment at: llvm/lib/MC/MCObjectFileInfo.cpp:1015
+  // At the moment the DXBC section should end up empty.
+  TextSection = Ctx->getDXContainerSection("DXBC", SectionKind::getText());
+}
----------------
What about the DXIL section, do you need to initialise it here, too?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127165



More information about the llvm-commits mailing list