[geany/geany] 650972: TravisCI: Remove requirement to fail build on aggregate return (#1740)

elextr git-noreply at xxxxx
Sat Jan 13 05:03:22 UTC 2018


Branch:      refs/heads/master
Author:      elextr <elextr at gmail.com>
Committer:   Matthew Brush <mbrush at codebrainz.ca>
Date:        Sat, 13 Jan 2018 05:03:22 UTC
Commit:      65097208df76439f0f059bafad5945624131475b
             https://github.com/geany/geany/commit/65097208df76439f0f059bafad5945624131475b

Log Message:
-----------
TravisCI: Remove requirement to fail build on aggregate return (#1740)

Returning large aggregates as values is a potential performance problem, but preventing all aggregate returns is inappropriate because:

1. returning small aggregates that are mostly handled as complete objects as return values often makes code clearer.

2. Modern (well not so modern actually) ABIs allow for small aggregates to be returned in registers, so forcing all returns to memory is actually likely to be slower.

3. if code review does not notice a large aggregate return and nobody notices a performance issue then it doesn't matter


Modified Paths:
--------------
    .travis.yml

Modified: .travis.yml
2 lines changed, 1 insertions(+), 1 deletions(-)
===================================================================
@@ -21,7 +21,7 @@ install:
   - sudo apt-get install -y --no-install-recommends doxygen
   - sudo apt-get install -y python-lxml
 before_script:
-  - export CFLAGS="-g -O2 -Werror=pointer-arith -Werror=aggregate-return -Werror=implicit-function-declaration"
+  - export CFLAGS="-g -O2 -Werror=pointer-arith -Werror=implicit-function-declaration"
 script:
   - NOCONFIGURE=1 ./autogen.sh
   - >



--------------
This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).


More information about the Commits mailing list