[PATCH] D40051: AMDGPU: Rename Bonaire target to be gfx704; update target feature handling

[Tony Tye via Phabricator via llvm-commits]

t-tye added inline comments.


================
Comment at: docs/AMDGPUUsage.rst:126
+     -----------------------------------------------------------------------------------
+     ``gfx700``  - ``kaveri``    ``amdgcn``   APU                     - A6-7000
+                                                                      - A6 Pro-7050B
----------------
kzhuravl wrote:
> Should this APU have an xnack on by default? I remember we talked about defaulting xnack to on for all APUs.
I do not believe hardware added XNACK support until GFX8.


================
Comment at: docs/AMDGPUUsage.rst:144
+                                                                      - Radeon R390x
+     ``gfx703``  - ``kabini``    ``amdgcn``   APU                     - E1-2100
+                 - ``mullins``                                        - E1-2200
----------------
kzhuravl wrote:
> Should this APU have an xnack on by default? I remember we talked about defaulting xnack to on for all APUs.
I do not believe hardware added XNACK support until GFX8.


https://reviews.llvm.org/D40051