Hahnfeld added a comment.

I didn't follow the recent CUDA support in Clang very closely, but this makes 
sense to me.



================
Comment at: clang/lib/Basic/Targets/NVPTX.cpp:48-49
     PTXVersion = llvm::StringSwitch<unsigned>(Feature)
+                     .Case("+ptx72", 74)
+                     .Case("+ptx71", 73)
                      .Case("+ptx72", 72)
----------------
Do you mean `+ptx74` and `+ptx73`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108239

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to