[PATCH] D21061: [ARM] Remove exit-on-error flag from test (PR27765)

Diana Picus via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 10 01:58:45 PDT 2016


rovka added a comment.

In http://reviews.llvm.org/D21061#453339, @rengolin wrote:

> In http://reviews.llvm.org/D21061#452291, @rovka wrote:
>
> > I can't find any tests for this error on other targets. Should I try to add one at least for X86?
>
>
> If you can, that'd be great. It would ease my worries about other arches.


Ok, working on it.

However, I think your concern is more general - on any test where we check for one error, we might want to make sure we don't trigger other bogus errors down the line. Should all our tests that check for one error also make sure there aren't any unexpected errors issued? I can put that in my backlog if it sounds like something we want.


http://reviews.llvm.org/D21061





More information about the llvm-commits mailing list