[PATCH] D122082: Add DXIL Bitcode Writer and DXIL testing

[Tom Stellard via Phabricator via llvm-commits]

tstellar added a comment.

In D122082#3397728 <https://reviews.llvm.org/D122082#3397728>, @beanz wrote:

> In D122082#3395747 <https://reviews.llvm.org/D122082#3395747>, @nikic wrote:
>
>> @xbolva00 Don't think the board has any relation to this, but I do agree that having some central documentation would be great. While many of the ongoing migration efforts are well-documented by themselves, there's currently no central place that collects them. I've created https://reviews.llvm.org/D122122 as a starting point, feedback welcome.
>
> @nikic, even that document doesn't say support for opaque pointers is a "hard requirement" for new contributions.
>
> Given that there is no documentation or clear community consensus anywhere that I can find that it should be a hard requirement, and that this code is going into an experimental backend, which (as documented in the support policy: https://llvm.org/docs/SupportPolicy.html) means "you break it, I fix it". Would you revise your "needs revision" feedback?

It would be great if there were more documentation around the migration timelines, but lack of documentation does not mean lack of plans or lack of consensus.  I would recommend starting a "Does the DXIL backend need to be compatible with Opaque Pointers" discussion on Discourse just so we can open this question up to a wider audience and get a little more clarity on this issue.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122082