arm: Use -O2 instead of -O as optimization flag
ClosedPublic
Actions

Authored by diizzy on Jul 21 2019, 10:37 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Nov 20, 8:28 PM

	Unknown Object (File)
	Thu, Nov 20, 8:27 PM

	Unknown Object (File)
	Thu, Nov 20, 8:25 PM

	Unknown Object (File)
	Thu, Nov 20, 8:16 PM

	Unknown Object (File)
	Thu, Nov 13, 12:44 AM

	Unknown Object (File)
	Sat, Nov 8, 2:38 AM

	Unknown Object (File)
	Tue, Oct 28, 1:21 AM

	Unknown Object (File)
	Sun, Oct 26, 11:42 AM

Subscribers

Details

Reviewers

manu
emaste

Commits

rS350210: arm: Use -O2 instead of -O as optimization flag

Summary

When using Clang -O is equivalent to -O2, change it -O2 to make it consistent with other platforms.
Reference: https://clang.llvm.org/docs/ClangCommandLineReference.html#optimization-level

Test Plan

Tested on armv7, Orange Pi PC

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

diizzy created this revision.Jul 21 2019, 10:37 PM

Herald added subscribers: emaste, imp, bdrewery. · View Herald TranscriptJul 21 2019, 10:37 PM

Seems reasonable. Was originally introduced in rS173375 but Clang is the default compiler and we can assume that contemporary GCC won't have "the -O2 issues."

This revision is now accepted and ready to land.Jul 22 2019, 1:42 AM

manu accepted this revision.Jul 22 2019, 10:15 AM

Closed by commit rS350210: arm: Use -O2 instead of -O as optimization flag (authored by manu). · Explain WhyJul 22 2019, 10:18 AM

This revision was automatically updated to reflect the committed changes.

manu added a commit: rS350210: arm: Use -O2 instead of -O as optimization flag.

Revision Contents
Changeset List

Path

Size

head/

share/

mk/

sys.mk

2 lines

Diff 60017

View Options

arm: Use -O2 instead of -O as optimization flagClosedPublicActions