alekseyshl added a comment. In https://reviews.llvm.org/D29660#838219, @gtbercea wrote:
> In https://reviews.llvm.org/D29660#838194, @arphaman wrote: > > > The failures were very widespread, e.g. there's a linux buildbot that was > > red until the revert: http://bb.pgr.jp/builders/test-clang-i686-linux-RA. > > If you have access to a linux machine you should be able to reproduce the > > failures that the bot experienced by using the same cmake arguments (I > > don't know the exact ones, but judging from the bot you should be able to > > reproduce them using 32 bit release build with assertions enabled). I don't > > know what GPU that buildbot has. > > > Thanks Alex, I will try to reproduce it locally. > > > I'll try to get the detailed test output for my local machine today as well. > > Oh that would be great! Thanks a lot! :) In https://reviews.llvm.org/D29660#838262, @gtbercea wrote: > 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 Our bots still fai after this changel: http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/7085 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
