[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