kzhuravl marked an inline comment as done.
kzhuravl added a comment.

In D149986#4334274 <https://reviews.llvm.org/D149986#4334274>, @Pierre-vh wrote:

> I think that if this is a new property of the GFX940/941 targets, and turning 
> it off shouldn't be possible, we shouldn't even bother with a feature and 
> just set a bool in the ST for those targets

AFAIK, the subtarget only knows the generation and has access to features. The 
subtarget cannot differentiate between gfx940, gfx941, gfx942.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149986

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D149... Matt Arsenault via Phabricator via cfe-commits
    • [PATCH]... Konstantin Zhuravlyov via Phabricator via cfe-commits
    • [PATCH]... Juan Manuel Martinez CaamaƱo via Phabricator via cfe-commits
    • [PATCH]... Konstantin Zhuravlyov via Phabricator via cfe-commits
    • [PATCH]... Pierre van Houtryve via Phabricator via cfe-commits
    • [PATCH]... Konstantin Zhuravlyov via Phabricator via cfe-commits
    • [PATCH]... Konstantin Zhuravlyov via Phabricator via cfe-commits
    • [PATCH]... Scott Linder via Phabricator via cfe-commits
    • [PATCH]... Matt Arsenault via Phabricator via cfe-commits

Reply via email to