[PATCH] D71358: AMDGPU: Remove denormal subtarget features

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 13 05:22:19 PST 2019


nhaehnle added a comment.

Would it be possible to do this patch in two steps:

1. First step: Add the parsing of the new function attribute, but keep the subtarget feature.
2. Second step: Remove the subtarget feature.

This would make the upgrade process easier for out-of-tree users (Mesa, LLPC).

Apart from that this seems like a good change to me.


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

https://reviews.llvm.org/D71358





More information about the llvm-commits mailing list