Correct force inline syntax for IAR compiler #11523
Conversation
The IAR compiler generates the following warning: |
| @@ -83,8 +83,11 @@ class Ticker : public TimerEvent, private NonCopyable<Ticker> { | |||
| * @param func pointer to the function to be called | |||
| * @param t the time between calls in seconds | |||
| */ | |||
| template <typename F> | |||
| MBED_FORCEINLINE void attach(F &&func, float t) | |||
| #if defined(__ICCARM__) | |||
There was a problem hiding this comment.
That's pretty grim, but it does appear to be the only forceinline template in the tree. I was thinking there were a bunch in the atomic stuff, but no.
I'd pull the void attach(...) out of the #if onto its own line.
There was a problem hiding this comment.
Unlinke other compilers supported, the IAR compiler requires the pre-processor extension to force inline a method to be placed before the keyword `template` if the method is declared with one.
hugueskamba
force-pushed
the
hk-correct-force-inline-syntax-for-iar
branch
from
f65df2e
to
e2aeb60
Compare
|
CI started |
|
CI internal problem in memory benchmark.. restarting the job |
Test run: FAILEDSummary: 1 of 11 test jobs failed Failed test jobs:
|
|
All green 👍 |
Description
The IAR compiler generates the following warning:
[Warning] Ticker.h@87,0: [Pe606]: this pragma must immediately precede a declarationUnlinke other compilers supported, the IAR compiler requires the
pre-processor extension to force inline a method to be placed before
the keyword
templateif the method is declared with one.Pull request type
Reviewers
@evedon
Release Notes