tra added a comment.

I'm not sure that's the problem worth solving.

Magically changing compiler target based on something external to compiler is a 
bad idea IMO. I would expect a compilation with exactly the same compiler 
options to do exactly the same thing. If we magically change default target, 
that will not be the case.

Also, there's no good default for a GPU. I can't think of anything that would 
work out of the box for most of the users. 
In practice compilation for the GPU require specifying GPU target set that's 
specific to particular user.

It may make more sense to just bump the default to a sensible value. E.g. 
`sm_60`, warn users ahead of time and flip the default at some point later. 
This will shift the default towards the target that's useful for most of the 
GPUs that are currently out there (though there are still a lot of sm_35 GPUs 
in the clouds, so it may be a reasonable default, too).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75811



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

Reply via email to