gtbercea added inline comments.
================ Comment at: lib/Driver/ToolChains/Cuda.h:90 - } }; ---------------- tra wrote: > gtbercea wrote: > > gtbercea wrote: > > > I would also like to keep the spirit of this code if not in this exact > > > form at least something that performs the same functionality. > > @tra what's your opinion on this code? Should this stay, stay but modified > > to be more robust or taken out completely? > There are currently no users for this. In general, I would rather not have > magically-changing default GPU based on how broken your CUDA installation is. > IMO it would be better to keep defaults static and fail if prerequisites are > not met. I would have thought that it is up to the compiler to select, as default, the lowest viable compute capability. This is what this code aims to do (whether it actually does that's a separate issue :) ). https://reviews.llvm.org/D38883 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
