arsenm added inline comments.
================
Comment at: llvm/lib/TargetParser/TargetParser.cpp:289
Features["image-insts"] = true;
+ Features["gds"] = true;
+ Features["gws"] = true;
----------------
yaxunl wrote:
> 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.
But it's easy to add back in if and when that happens
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158367/new/
https://reviews.llvm.org/D158367
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
