On Mon, 2020-03-09 at 22:20 +0000, Andrea Corallo wrote:
> Hi all,
> 
> second version of the patch for the 200 characters limit for literal
> strings addressing comments.
> 
> make check-jit is passing clean.
> 
> Best Regards
>   Andrea
> 
> gcc/jit/ChangeLog
> 2020-??-??  Andrea Corallo  <andrea.cora...@arm.com>
> 
>       * jit-playback.h
>       (gcc::jit::playback::context m_recording_ctxt): Remove
>       m_char_array_type_node field.
>       * jit-playback.c
>       (playback::context::context) Remove m_char_array_type_node from
> member
>       initializer list.
>       (playback::context::new_string_literal) Fix logic to handle
> string
>       length > 200.
> 
> gcc/testsuite/ChangeLog
> 2020-??-??  Andrea Corallo  <andrea.cora...@arm.com>
> 
>       * jit.dg/all-non-failing-tests.h: Add test-long-string-
> literal.c.
>       * jit.dg/test-long-string-literal.c: New testcase.
> 

diff --git a/gcc/testsuite/jit.dg/all-non-failing-tests.h 
b/gcc/testsuite/jit.dg/all-non-failing-tests.h
index 0272e6f846f..1b3d5618779 100644
--- a/gcc/testsuite/jit.dg/all-non-failing-tests.h
+++ b/gcc/testsuite/jit.dg/all-non-failing-tests.h
@@ -220,6 +220,13 @@
 #undef create_code
 #undef verify_code
 
+/* test-long-string-literal.c */
+#define create_code create_code_long_string_literal
+#define verify_code verify_code_long_string_literal
+#include "test-long-string-literal.c"
+#undef create_code
+#undef verify_code
+

Please adjust where you add this so that these things are in
alphabetical order.

An entry also needs to be added to the "testcases" array at the end of
the header for this to do anything.

OK with that change.

David

Reply via email to