Ticket #571 (closed patch: fixed)

Opened 5 years ago

Last modified 5 years ago

[PATCH] Enable optimizations in CFLAGS for gcc

Reported by: doughera Owned by:
Priority: normal Milestone:
Component: configure Version:
Severity: low Keywords:
Cc: Language:
Patch status: applied Platform:

Description

Currently, CFLAGS.in disables optimizations for core_ops_cg.c and core_ops_switch.c, but not for core_ops_cgp.c. Disabling optimization is not necessary for gcc, but other compilers might need it for those two and for core_ops_cgp.c as well.

This patch makes a small step towards regularlizing CFLAGS.in. If using gcc, optimization is allowed on all three. If not using gcc, optimization is disabled on all three files. This seems to be the safest general starting assumption. If other compilers are able to compile those files with optimization, then the #UNLESS statements could be changed to accommodate those as well.

Of course all this can be terribly dependent on the platform, compiler version, available resources, and particular optimization switches chosen. There is, unfortunately, no obvious easy way to generalize this. However, I think it's a good step forward.

Attachments

tt571-cflags.patch Download (1.0 KB) - added by doughera 5 years ago.

Change History

Changed 5 years ago by doughera

Changed 5 years ago by Infinoid

  • status changed from new to closed
  • resolution set to fixed
  • patch set to applied

Thanks, applied as r38265.

Note: See TracTickets for help on using tickets.