shiltian wrote:

Isn't offload bundler a target agnostic tool?

> This could help us not break things internally, and then as tools and APIs 
> update to V6, they can also update to 4-field triples

PSDB has full green. I also created issues to all repos that use 
`clang-offload-bundler` binary directly. Most of them have responded and 
started using 4-tuple now. IMHO if we tell users, hey, if you do COV5, you can 
use 3- or 4-tuple, but if you do COV6, you have to use 4-tuple, it's gonna me 
way more confusing than directly enforcing them. In addition, internally we 
have switched to COV6 by default already for staging branch, where this PR is 
gonna go, and will be soon in mainline.

https://github.com/llvm/llvm-project/pull/122629
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to