Messages in this thread | | | From | Nick Desaulniers <> | Date | Tue, 24 Mar 2020 09:43:26 -0700 | Subject | Re: [PATCH 02/16] x86: remove unneeded defined(__ASSEMBLY__) check from asm/dwarf2.h |
| |
On Tue, Mar 24, 2020 at 1:49 AM Masahiro Yamada <masahiroy@kernel.org> wrote: > > This header file has the following check at the top: > > #ifndef __ASSEMBLY__ > #warning "asm/dwarf2.h should be only included in pure assembly files" > #endif > > So, we expect defined(__ASSEMBLY__) is always true. > > Signed-off-by: Masahiro Yamada <masahiroy@kernel.org> > Acked-by: Jason A. Donenfeld <Jason@zx2c4.com>
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
> --- > > arch/x86/include/asm/dwarf2.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/x86/include/asm/dwarf2.h b/arch/x86/include/asm/dwarf2.h > index ae391f609840..5a0502212bc5 100644 > --- a/arch/x86/include/asm/dwarf2.h > +++ b/arch/x86/include/asm/dwarf2.h > @@ -36,7 +36,7 @@ > #define CFI_SIGNAL_FRAME > #endif > > -#if defined(CONFIG_AS_CFI_SECTIONS) && defined(__ASSEMBLY__) > +#if defined(CONFIG_AS_CFI_SECTIONS) > #ifndef BUILD_VDSO > /* > * Emit CFI data in .debug_frame sections, not .eh_frame sections. > -- > 2.17.1 > > -- > You received this message because you are subscribed to the Google Groups "Clang Built Linux" group. > To unsubscribe from this group and stop receiving emails from it, send an email to clang-built-linux+unsubscribe@googlegroups.com. > To view this discussion on the web visit https://groups.google.com/d/msgid/clang-built-linux/20200324084821.29944-3-masahiroy%40kernel.org.
-- Thanks, ~Nick Desaulniers
| |