[PATCH] D158367: [AMDGPU] Add target feature gds/gws to clang

[Yaxun Liu via Phabricator via cfe-commits]

yaxunl added inline comments.


================
Comment at: llvm/lib/TargetParser/TargetParser.cpp:289
       Features["image-insts"] = true;
+      Features["gds"] = true;
+      Features["gws"] = true;
----------------
arsenm wrote:
> Gds feature is unused 
I am thinking to keep it in case we need it for newly added builtins or want to diagnose ops requires gds.


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

https://reviews.llvm.org/D158367