tra added a comment.

In http://reviews.llvm.org/D20985#448822, @jlebar wrote:

> How is this different from test/SemaCUDA/launch_bounds.cu:27-28?  It does
>
>   const int constint = 512;
>   __launch_bounds__(constint) void TestConstInt(void);
>   
>
> which looks verbatim the same as this testcase.


Existing test is a declaration of the function which did not trigger the crash.
Second issue is that -verify interferes with reproduction case -- the crash 
does not happen if any //expect-* are seen before it.
Plus, the outcome of the failing test is a crash which would prevent reports of 
other failures.
Separate test file makes the crash isolated and reliably reproducible.


http://reviews.llvm.org/D20985



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to