Re: [edk2-devel] [PATCH 1/1] DynamicTablesPkg: include ARM intrinsics library to fix 32-bit build

["Philippe Mathieu-Daudé" <philmd@redhat.com>]

On 10/15/19 1:07 PM, Ard Biesheuvel wrote:
> DynamicTablesPkg can be built for ARM as well as for AARCH64, but on
> the former, doing so will result in a build failure due to the lack
> of 64-bit division helpers provided by the ArmPkg intrinsics library.
> So add the missing reference, for both ARM and AARCH64 (which may
> start relying on intrinsics due to future changes)
> 
> Link: https://bugzilla.tianocore.org/show_bug.cgi?id=2269
> Reported-by: Laszlo Ersek <lersek@redhat.com>
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> ---
>   DynamicTablesPkg/DynamicTablesPkg.dsc | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/DynamicTablesPkg/DynamicTablesPkg.dsc b/DynamicTablesPkg/DynamicTablesPkg.dsc
> index ef958077ed48..19beaaf370f8 100644
> --- a/DynamicTablesPkg/DynamicTablesPkg.dsc
> +++ b/DynamicTablesPkg/DynamicTablesPkg.dsc
> @@ -32,6 +32,7 @@ [LibraryClasses]
>     UefiDriverEntryPoint|MdePkg/Library/UefiDriverEntryPoint/UefiDriverEntryPoint.inf
>   
>   [LibraryClasses.ARM, LibraryClasses.AARCH64]
> +  NULL|ArmPkg/Library/CompilerIntrinsicsLib/CompilerIntrinsicsLib.inf
>     PL011UartLib|ArmPlatformPkg/Library/PL011UartLib/PL011UartLib.inf
>   
>   [Components.common]
> 

Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>