[PATCH] D85882: [AMDGPU] Update subtarget features for new target ID support

[Austin Kerbow via Phabricator via llvm-commits]

kerbowa added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPU.td:839
    FeatureAtomicFaddInsts,
+   FeatureXNACK,
    FeatureSRAMECC,
----------------
arsenm wrote:
> This shouldn't be on any target definitions
It's intentional in order to keep the current defaults. Targets with FeatureXNACK are ones that used to be default xnack on.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85882