gtbercea added a comment. In https://reviews.llvm.org/D29660#838185, @alekseyshl wrote:
> In https://reviews.llvm.org/D29660#838150, @gtbercea wrote: > > > First of all, I apologize if I've upset you with my previous post. I am > > actively working on understanding what is causing these issues. It is not > > my intention to write tests that work on local configurations only. I am > > upset to see that these tests keep failing for your and maybe other > > configurations. Without knowing the actual reason of the failures I can > > only speculate what is going wrong with them hence the flurry of changes. > > > Thank you, apology accepted. That was exactly my point, not to start a fight, > but to emphasize that depending on local configuration is never going to > work, you will never be able to see and test all of them. Please disable the > test ASAP and until the better way to handle it is determined. I have disabled all the offloading tests apart from the ones that pertain to the patch previous to the one introducing Cubin integration into host bin. Please let me know if you see any more failures on your side. If you do feel free to revert all the patches up to and including: https://reviews.llvm.org/D29654 310625 Repository: rL LLVM https://reviews.llvm.org/D29660 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
