[llvm] r209309 - MC: loosen an overzealous assertion
Saleem Abdulrasool
compnerd at compnerd.org
Thu May 22 20:05:17 PDT 2014
On Thu, May 22, 2014 at 8:47 AM, Rafael EspĂndola <
rafael.espindola at gmail.com> wrote:
> >> Why does this test require assertions?
> >
> >
> > The test is to ensure that the driver doesn't crash due to the assertion.
> > The actual handling of it wasn't changed. I could be convinced that the
> > test doesn't add much though.
>
> I think that the point is valid with and without asserts, so the
> REQUIRES line can be removed.
>
Yeah, you convinced me that I wasnt thinking about this properly.
Addressed with SVN r209496.
> Cheers,
> Rafael
--
Saleem Abdulrasool
compnerd (at) compnerd (dot) org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140522/e61e817e/attachment.html>
More information about the llvm-commits
mailing list