tra marked an inline comment as done. tra added inline comments.
================ Comment at: clang/lib/Basic/Cuda.cpp:61 + // clang-format off + SM2(20, "compute_20"), SM2(21, "compute_20"), // Fermi + SM(30), SM(32), SM(35), SM(37), // Kepler ---------------- yaxunl wrote: > Thanks for the efforts. Really appreciate this. > > Can we separate this part to a def file and include it here and also in enum > class CudaArch? Then we only need to maintain the def file. > > Thanks. If we had similar per-GPU code in many places, then it would make a lot of sense. At the moment I'm not sure it would be worth it -- I don't see any other code that would benefit, other than this file. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77665/new/ https://reviews.llvm.org/D77665 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits