https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90340

--- Comment #6 from Martin Liška <marxin at gcc dot gnu.org> ---
(In reply to Fredrik Hederstierna from comment #5)
> I use patched sources from
> http://gcc.hederstierna.com/csibe

I've just tried that but I see quite some compilation errors:

/home/marxin/Downloads/CSiBE-v2.1.1-patched2/CSiBE/./src/./flex-2.5.31/../include/inttypes.h:12:26:
error: conflicting types for ‘int64_t’
/home/marxin/Downloads/CSiBE-v2.1.1-patched2/CSiBE/./src/./libmspack/test/../../include/inttypes.h:12:26:
error: conflicting types for ‘int64_t’
/home/marxin/Downloads/CSiBE-v2.1.1-patched2/CSiBE/./src/./mpeg2dec-0.3.1/libmpeg2/../../include/inttypes.h:12:26:
error: conflicting types for ‘int64_t’

> 
> I think you could download and try it out.
> Toolchain I build with
> https://github.com/fredrikhederstierna/buildbuddy
> 
> Otherwise I think there are at least two more code size regressions open for
> gcc-9.1.0 on ARM against CSiBE, so maybe there are already some CSiBE test
> setup running that could be quicker path forward.
> 
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90249
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90255
> 
> Code size regressions on CSiBE for gcc-9.1.0 I think is worked on.
> Thanks/Fredrik

I'm curious how much can save following patch:

diff --git a/gcc/tree-switch-conversion.h b/gcc/tree-switch-conversion.h
index b3bc4b9ddf7..904827fa040 100644
--- a/gcc/tree-switch-conversion.h
+++ b/gcc/tree-switch-conversion.h
@@ -271,7 +271,7 @@ struct jump_table_cluster: public group_cluster
   static bool is_enabled (void);

   /* Max growth ratio for code that is optimized for size.  */
-  static const unsigned HOST_WIDE_INT max_ratio_for_size = 3;
+  static const unsigned HOST_WIDE_INT max_ratio_for_size = 2;

   /* Max growth ratio for code that is optimized for speed.  */
   static const unsigned HOST_WIDE_INT max_ratio_for_speed = 8;

Reply via email to