arsenm accepted this revision.
arsenm added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPU.td:1231-1261
+  [FeatureGFX9,
+   FeatureGFX90AInsts,
+   FeatureGFX940Insts,
+   FeatureFmaMixInsts,
+   FeatureLDSBankCount32,
+   FeatureDLInsts,
+   FeatureFmacF64Inst,
----------------
We probably should make better use of groups and implied features to cut down 
on the size of these lists for each of these 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149982

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D149982: AMD... Konstantin Zhuravlyov via Phabricator via cfe-commits
    • [PATCH] D149982... Matt Arsenault via Phabricator via cfe-commits
    • [PATCH] D149982... Konstantin Zhuravlyov via Phabricator via cfe-commits

Reply via email to