| Message ID | 20230620083918.9134-1-frederic.petrot@univ-grenoble-alpes.fr |
|---|---|
| State | New |
| Headers | show |
| Series | x86_64/atomic128-ldst.h: fix arch include guard | expand |
On 6/20/23 10:39, Frédéric Pétrot wrote: > Fix arch name in the define that prevents multiple inclusions. > > Signed-off-by: Frédéric Pétrot <frederic.petrot@univ-grenoble-alpes.fr> > --- > host/include/x86_64/host/atomic128-ldst.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) Whoopsie. Reviewed-by: Richard Henderson <richard.henderson@linaro.org> r~ > > diff --git a/host/include/x86_64/host/atomic128-ldst.h b/host/include/x86_64/host/atomic128-ldst.h > index adc9332f91..d0240d4212 100644 > --- a/host/include/x86_64/host/atomic128-ldst.h > +++ b/host/include/x86_64/host/atomic128-ldst.h > @@ -8,8 +8,8 @@ > * atomic primitive is meant to provide. > */ > > -#ifndef AARCH64_ATOMIC128_LDST_H > -#define AARCH64_ATOMIC128_LDST_H > +#ifndef X86_64_ATOMIC128_LDST_H > +#define X86_64_ATOMIC128_LDST_H > > #ifdef CONFIG_INT128_TYPE > #include "host/cpuinfo.h" > @@ -65,4 +65,4 @@ static inline void atomic16_set(Int128 *ptr, Int128 val) > #include "host/include/generic/host/atomic128-ldst.h" > #endif > > -#endif /* AARCH64_ATOMIC128_LDST_H */ > +#endif /* X86_64_ATOMIC128_LDST_H */
On 20/6/23 10:39, Frédéric Pétrot wrote: > Fix arch name in the define that prevents multiple inclusions. > > Signed-off-by: Frédéric Pétrot <frederic.petrot@univ-grenoble-alpes.fr> > --- > host/include/x86_64/host/atomic128-ldst.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
On 6/20/23 10:39, Frédéric Pétrot wrote: > Fix arch name in the define that prevents multiple inclusions. > > Signed-off-by: Frédéric Pétrot <frederic.petrot@univ-grenoble-alpes.fr> > --- > host/include/x86_64/host/atomic128-ldst.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/host/include/x86_64/host/atomic128-ldst.h b/host/include/x86_64/host/atomic128-ldst.h > index adc9332f91..d0240d4212 100644 > --- a/host/include/x86_64/host/atomic128-ldst.h > +++ b/host/include/x86_64/host/atomic128-ldst.h > @@ -8,8 +8,8 @@ > * atomic primitive is meant to provide. > */ > > -#ifndef AARCH64_ATOMIC128_LDST_H > -#define AARCH64_ATOMIC128_LDST_H > +#ifndef X86_64_ATOMIC128_LDST_H > +#define X86_64_ATOMIC128_LDST_H > > #ifdef CONFIG_INT128_TYPE > #include "host/cpuinfo.h" > @@ -65,4 +65,4 @@ static inline void atomic16_set(Int128 *ptr, Int128 val) > #include "host/include/generic/host/atomic128-ldst.h" > #endif > > -#endif /* AARCH64_ATOMIC128_LDST_H */ > +#endif /* X86_64_ATOMIC128_LDST_H */ Actually, I recently merged a fix for this: 8a2bd747877. r~
diff --git a/host/include/x86_64/host/atomic128-ldst.h b/host/include/x86_64/host/atomic128-ldst.h index adc9332f91..d0240d4212 100644 --- a/host/include/x86_64/host/atomic128-ldst.h +++ b/host/include/x86_64/host/atomic128-ldst.h @@ -8,8 +8,8 @@ * atomic primitive is meant to provide. */ -#ifndef AARCH64_ATOMIC128_LDST_H -#define AARCH64_ATOMIC128_LDST_H +#ifndef X86_64_ATOMIC128_LDST_H +#define X86_64_ATOMIC128_LDST_H #ifdef CONFIG_INT128_TYPE #include "host/cpuinfo.h" @@ -65,4 +65,4 @@ static inline void atomic16_set(Int128 *ptr, Int128 val) #include "host/include/generic/host/atomic128-ldst.h" #endif -#endif /* AARCH64_ATOMIC128_LDST_H */ +#endif /* X86_64_ATOMIC128_LDST_H */
Fix arch name in the define that prevents multiple inclusions. Signed-off-by: Frédéric Pétrot <frederic.petrot@univ-grenoble-alpes.fr> --- host/include/x86_64/host/atomic128-ldst.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)