{"action":"created","issue":{"url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/7247","repository_url":"https://api.github.com/repos/ARMmbed/mbed-os","labels_url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/7247/labels{/name}","comments_url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/7247/comments","events_url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/7247/events","html_url":"https://github.com/ARMmbed/mbed-os/pull/7247","id":333396888,"node_id":"MDExOlB1bGxSZXF1ZXN0MTk1NjE5NDE2","number":7247,"title":"Tools: Check compiler version","user":{"login":"theotherjimmy","id":685409,"node_id":"MDQ6VXNlcjY4NTQwOQ==","avatar_url":"https://avatars2.githubusercontent.com/u/685409?v=4","gravatar_id":"","url":"https://api.github.com/users/theotherjimmy","html_url":"https://github.com/theotherjimmy","followers_url":"https://api.github.com/users/theotherjimmy/followers","following_url":"https://api.github.com/users/theotherjimmy/following{/other_user}","gists_url":"https://api.github.com/users/theotherjimmy/gists{/gist_id}","starred_url":"https://api.github.com/users/theotherjimmy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/theotherjimmy/subscriptions","organizations_url":"https://api.github.com/users/theotherjimmy/orgs","repos_url":"https://api.github.com/users/theotherjimmy/repos","events_url":"https://api.github.com/users/theotherjimmy/events{/privacy}","received_events_url":"https://api.github.com/users/theotherjimmy/received_events","type":"User","site_admin":false},"labels":[{"id":965671858,"node_id":"MDU6TGFiZWw5NjU2NzE4NTg=","url":"https://api.github.com/repos/ARMmbed/mbed-os/labels/release-version:%205.9.2","name":"release-version: 5.9.2","color":"fef2c0","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":28,"created_at":"2018-06-18T19:06:21Z","updated_at":"2018-06-28T12:55:00Z","closed_at":"2018-06-27T23:42:50Z","author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/ARMmbed/mbed-os/pulls/7247","html_url":"https://github.com/ARMmbed/mbed-os/pull/7247","diff_url":"https://github.com/ARMmbed/mbed-os/pull/7247.diff","patch_url":"https://github.com/ARMmbed/mbed-os/pull/7247.patch"},"body":"### Description\n\nIn the past, we allowed any version of any toolchain. That will be a \nproblem if/when we upgrade to ARM Compiler 5.10 or IAR 8. This PR\nhas the tools enforce the version of the compiler on `mbed compile`,\npreventing any issues with compiler version\n\n### Pull request type\n\n    [x] Fix\n    [ ] Refactor\n    [ ] New target\n    [ ] Feature\n    [ ] Breaking change"},"comment":{"url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/comments/401024425","html_url":"https://github.com/ARMmbed/mbed-os/pull/7247#issuecomment-401024425","issue_url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/7247","id":401024425,"node_id":"MDEyOklzc3VlQ29tbWVudDQwMTAyNDQyNQ==","user":{"login":"mattbrown015","id":26009918,"node_id":"MDQ6VXNlcjI2MDA5OTE4","avatar_url":"https://avatars0.githubusercontent.com/u/26009918?v=4","gravatar_id":"","url":"https://api.github.com/users/mattbrown015","html_url":"https://github.com/mattbrown015","followers_url":"https://api.github.com/users/mattbrown015/followers","following_url":"https://api.github.com/users/mattbrown015/following{/other_user}","gists_url":"https://api.github.com/users/mattbrown015/gists{/gist_id}","starred_url":"https://api.github.com/users/mattbrown015/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mattbrown015/subscriptions","organizations_url":"https://api.github.com/users/mattbrown015/orgs","repos_url":"https://api.github.com/users/mattbrown015/repos","events_url":"https://api.github.com/users/mattbrown015/events{/privacy}","received_events_url":"https://api.github.com/users/mattbrown015/received_events","type":"User","site_admin":false},"created_at":"2018-06-28T12:55:00Z","updated_at":"2018-06-28T12:55:00Z","author_association":"NONE","body":"I wondered if the difference would diminish in a release build, but apparently not.\r\n\r\nThe following is from a release build for STM32L4.\r\n\r\n6 2017-q2-update:\r\n\r\n    | [lib]\\c.a                           |  36555 |  2472 |    89 |\r\n    | [lib]\\stdc++.a                      |   9866 |    44 |   204 |\r\n    Total Static RAM memory (data + bss): 28992 bytes\r\n    Total Flash memory (text + data): 193458 bytes\r\n\r\n7 2017-q4-major:\r\n\r\n    | [lib]\\c.a                           |  33623 |  2472 |    89 |\r\n    | [lib]\\stdc++.a                      |   8285 |    12 |    44 |\r\n    Total Static RAM memory (data + bss): 28808 bytes\r\n    Total Flash memory (text + data): 187783 bytes\r\n\r\nDifference:\r\n\r\n    | [lib]\\c.a                           |   2932 |     0 |     0 |\r\n    | [lib]\\stdc++.a                      |   1581 |    32 |   160 |\r\n    Static RAM memory: 184 bytes\r\n    Flash memory: 5675 bytes\r\n\r\nI highlighted `c.a` and `stdc++.a` because they have the largest differences but actually the `.text` sections for most of the modules are different.  Some of the modules were bigger with '7 2017-q4-major' but most were smaller.\r\n\r\nI wonder if there is a significant default flag change but I haven't looked at the compiler release notes and, while I find this interesting, I've got more important things to be doing!\r\n\r\nWe shall probably follow your advice and use the _supported_ compiler."}}
{"actor":{"display_login":"mattbrown015"}}