alekseyshl added a comment.
In https://reviews.llvm.org/D29660#838305, @alekseyshl wrote:
> 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:
r310640 disables this test on Linux
cfe-commits mailing list