Re: [PATCH kernel] powerpc/llvm/lto: Allow LLVM LTO builds
From: Nick Desaulniers <hidden>
Date: 2022-05-03 21:24:57
Also in:
llvm
From: Nick Desaulniers <hidden>
Date: 2022-05-03 21:24:57
Also in:
llvm
On Thu, Apr 28, 2022 at 11:46 PM Alexey Kardashevskiy [off-list ref] wrote:
diff --git a/arch/powerpc/kernel/exceptions-64s.S b/arch/powerpc/kernel/exceptions-64s.S index b66dd6f775a4..5b783bd51260 100644 --- a/arch/powerpc/kernel/exceptions-64s.S +++ b/arch/powerpc/kernel/exceptions-64s.S@@ -476,9 +476,11 @@ DEFINE_FIXED_SYMBOL(\name\()_common_real, text) .if IHSRR_IF_HVMODE BEGIN_FTR_SECTION bne masked_Hinterrupt + b 4f FTR_SECTION_ELSE
Do you need to have the ELSE even if there's nothing in it; should it have a nop? The rest of the assembler changes LGTM, but withholding RB tag until we have Kconfig dependencies in better shape.
- bne masked_interrupt
ALT_FTR_SECTION_END_IFSET(CPU_FTR_HVMODE | CPU_FTR_ARCH_206)
+ bne masked_interrupt
+4:
.elseif IHSRR
bne masked_Hinterrupt
.else-- Thanks, ~Nick Desaulniers