Refs #21358: Limit gnu::optimize to Ubuntu

Javier Gil Aviles · Javier Gil Aviles · commit 0d8635061c31 · 2025-07-01T15:58:10.000+02:00
Signed-off-by: Javier Gil Aviles &lt;javiergil@eprosima.com&gt;
diff --git a/src/cpp/rtps/flowcontrol/FlowControllerImpl.hpp b/src/cpp/rtps/flowcontrol/FlowControllerImpl.hpp
@@ -126,7 +126,10 @@ struct FlowQueue
         }
 
         // TODO: remove optimization with GCC > 15
-        [[gnu::optimize("no-delete-null-pointer-checks")]] bool is_empty() const noexcept
+        #if defined(__linux__)
+        [[gnu::optimize("no-delete-null-pointer-checks")]]
+        #endif // if defined(__GNUC__)
+        bool is_empty() const noexcept
         {
             assert((&tail == head.writer_info.next && &head == tail.writer_info.previous) ||
                     (&tail != head.writer_info.next && &head != tail.writer_info.previous));

Original file line number	Diff line number	Diff line change
`@@ -126,7 +126,10 @@ struct FlowQueue`
`126`	`126`	`}`
`127`	`127`
`128`	`128`	`// TODO: remove optimization with GCC > 15`
`129`		`- [[gnu::optimize("no-delete-null-pointer-checks")]] bool is_empty() const noexcept`
	`129`	`+ #if defined(__linux__)`
	`130`	`+ [[gnu::optimize("no-delete-null-pointer-checks")]]`
	`131`	`+ #endif // if defined(__GNUC__)`
	`132`	`+ bool is_empty() const noexcept`
`130`	`133`	`{`
`131`	`134`	`assert((&tail == head.writer_info.next && &head == tail.writer_info.previous) \|\|`
`132`	`135`	`(&tail != head.writer_info.next && &head != tail.writer_info.previous));`