[PATCH] D75913: [AMDGPU] Use progbits type for .AMDGPU.disasm section

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 11 04:30:38 PDT 2020


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

In D75913#1915455 <https://reviews.llvm.org/D75913#1915455>, @arsenm wrote:

> Can we finally just remove this? There's a steady stream of issues from having this feature at all, and we really don't need it


I agree. Let's figure this out separately though. This change is a hack (sorry Tony), but I think it's basically fine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75913





More information about the llvm-commits mailing list