Arthur Heymans has submitted this change. ( https://review.coreboot.org/c/coreboot/+/84008?usp=email )
(
3 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: libpayload: Use defined length movs ......................................................................
libpayload: Use defined length movs
This fixes building with clang.
Change-Id: I2fabe7fbe3f8afac5c1128debf2e09a484f26fc5 Signed-off-by: Arthur Heymans arthur@aheymans.xyz Reviewed-on: https://review.coreboot.org/c/coreboot/+/84008 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Nico Huber nico.h@gmx.de --- M payloads/libpayload/arch/x86/pt.S 1 file changed, 4 insertions(+), 4 deletions(-)
Approvals: build bot (Jenkins): Verified Nico Huber: Looks good to me, approved
diff --git a/payloads/libpayload/arch/x86/pt.S b/payloads/libpayload/arch/x86/pt.S index 9b085e5..0e6e7bf 100644 --- a/payloads/libpayload/arch/x86/pt.S +++ b/payloads/libpayload/arch/x86/pt.S @@ -97,8 +97,8 @@ mov $main_page_table, %esi
loop_2mb: - mov %eax, (%rsi, %rcx, 8) - mov $0, 4(%rsi, %rcx, 8) + movl %eax, (%rsi, %rcx, 8) + movl $0, 4(%rsi, %rcx, 8) add $0x200000, %eax inc %ecx cmp %edi, %ecx @@ -111,8 +111,8 @@ mov $extra_page_table, %esi
fill_extra_page_table: - mov %eax, (%rsi, %rcx, 8) - mov $0, 4(%rsi, %rcx, 8) + movl %eax, (%rsi, %rcx, 8) + movl $0, 4(%rsi, %rcx, 8) add $4096, %eax inc %ecx cmp %edi, %ecx