This is an automated email from Gerrit.

"Antonio Borneo <borneo.anto...@gmail.com>" just uploaded a new patch set to 
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/6881

-- gerrit

commit c83da2ac864d0db64da0305856faa31e7360ada3
Author: Antonio Borneo <borneo.anto...@gmail.com>
Date:   Sat Mar 19 10:56:01 2022 +0100

    breakpoints: fix build on -fno-inline
    
    Some configuration of GCC could default to -fno-inline, causing
    the build to fail after commit fb43f1ff4e2f ("target: Rework 'set'
    variable of break-/watchpoints").
    
    Switch the new inline functions to 'static inline', as it's widely
    used in the rest of the code.
    
    Change-Id: I8bf31045a137bd34ed825f4b2a9338eb3c70046d
    Signed-off-by: Antonio Borneo <borneo.anto...@gmail.com>
    Fixes: fb43f1ff4e2f ("target: Rework 'set' variable of break-/watchpoints")

diff --git a/src/target/breakpoints.h b/src/target/breakpoints.h
index b97be718c0..0a5949594d 100644
--- a/src/target/breakpoints.h
+++ b/src/target/breakpoints.h
@@ -71,7 +71,7 @@ void breakpoint_remove_all(struct target *target);
 
 struct breakpoint *breakpoint_find(struct target *target, target_addr_t 
address);
 
-inline void breakpoint_hw_set(struct breakpoint *breakpoint, unsigned int 
hw_number)
+static inline void breakpoint_hw_set(struct breakpoint *breakpoint, unsigned 
int hw_number)
 {
        breakpoint->is_set = true;
        breakpoint->number = hw_number;
@@ -87,7 +87,7 @@ void watchpoint_remove(struct target *target, target_addr_t 
address);
 int watchpoint_hit(struct target *target, enum watchpoint_rw *rw,
                target_addr_t *address);
 
-inline void watchpoint_set(struct watchpoint *watchpoint, unsigned int number)
+static inline void watchpoint_set(struct watchpoint *watchpoint, unsigned int 
number)
 {
        watchpoint->is_set = true;
        watchpoint->number = number;

-- 

Reply via email to