[riot-notifications] [RIOT-OS/RIOT] cpu/cortexm_common: add inlined header only def for irq_% (#13999)

Francisco Acosta notifications at github.com
Mon May 4 13:57:17 CEST 2020


@kYc0o commented on this pull request.



>      __enable_irq();
-    return __get_PRIMASK();
+    return result;

What's the actual difference here regarding the original code? By inlining the function it is the "extra" variable required? By looking at the sizes it's obviously not affecting, just a question out of curiosity.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/13999#pullrequestreview-404919644
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200504/b3b1a55a/attachment.htm>


More information about the notifications mailing list