* [PATCH v3 11/12] CryptoPkg/IntrinsicLib: Remove .S files for IA32 arch
@ 2019-03-29 7:31 Shenglei Zhang
2019-03-29 8:33 ` Wang, Jian J
0 siblings, 1 reply; 5+ messages in thread
From: Shenglei Zhang @ 2019-03-29 7:31 UTC (permalink / raw)
To: edk2-devel; +Cc: Ting Ye, Gang Wei, Jian Wang
.nasm file has been added for X86 arch. .S assembly code
is not required any more.
https://bugzilla.tianocore.org/show_bug.cgi?id=1594
Cc: Ting Ye <ting.ye@intel.com>
Cc: Gang Wei <gang.wei@intel.com>
Cc: Jian Wang <jian.j.wang@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
---
.../Library/IntrinsicLib/Ia32/MathLShiftS64.S | 62 -----------------
.../Library/IntrinsicLib/Ia32/MathRShiftU64.S | 66 -------------------
.../Library/IntrinsicLib/IntrinsicLib.inf | 2 -
3 files changed, 130 deletions(-)
delete mode 100644 CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
delete mode 100644 CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
diff --git a/CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S b/CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
deleted file mode 100644
index 7031a59a71..0000000000
--- a/CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
+++ /dev/null
@@ -1,62 +0,0 @@
-#------------------------------------------------------------------------------
-#
-# Copyright (c) 2014, Intel Corporation. All rights reserved.<BR>
-# This program and the accompanying materials
-# are licensed and made available under the terms and conditions of the BSD License
-# which accompanies this distribution. The full text of the license may be found at
-# http://opensource.org/licenses/bsd-license.php.
-#
-# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
-# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
-#
-# Module Name:
-#
-# MathLShiftS64.S
-#
-# Abstract:
-#
-# 64-bit Math Worker Function.
-# Shifts a 64-bit signed value left by a certain number of bits.
-#
-#------------------------------------------------------------------------------
-
- .686:
- .code:
-
-ASM_GLOBAL ASM_PFX(__ashldi3)
-
-#------------------------------------------------------------------------------
-#
-# void __cdecl __ashldi3 (void)
-#
-#------------------------------------------------------------------------------
-ASM_PFX(__ashldi3):
- #
- # Handle shifting of 64 or more bits (return 0)
- #
- cmpb $64, %cl
- jae ReturnZero
-
- #
- # Handle shifting of between 0 and 31 bits
- #
- cmpb $32, %cl
- jae More32
- shld %cl, %eax, %edx
- shl %cl, %eax
- ret
-
- #
- # Handle shifting of between 32 and 63 bits
- #
-More32:
- movl %eax, %edx
- xor %eax, %eax
- and $31, %cl
- shl %cl, %edx
- ret
-
-ReturnZero:
- xor %eax, %eax
- xor %edx, %edx
- ret
diff --git a/CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S b/CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
deleted file mode 100644
index 24142b088e..0000000000
--- a/CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
+++ /dev/null
@@ -1,66 +0,0 @@
-#------------------------------------------------------------------------------
-#
-# Copyright (c) 2014, Intel Corporation. All rights reserved.<BR>
-# This program and the accompanying materials
-# are licensed and made available under the terms and conditions of the BSD License
-# which accompanies this distribution. The full text of the license may be found at
-# http://opensource.org/licenses/bsd-license.php.
-#
-# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
-# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
-#
-# Module Name:
-#
-# MathRShiftU64.S
-#
-# Abstract:
-#
-# 64-bit Math Worker Function.
-# Shifts a 64-bit unsigned value right by a certain number of bits.
-#
-#------------------------------------------------------------------------------
-
-
- .686:
- .code:
-
-ASM_GLOBAL ASM_PFX(__ashrdi3)
-
-#------------------------------------------------------------------------------
-#
-# void __cdecl __ashrdi3 (void)
-#
-#------------------------------------------------------------------------------
-ASM_PFX(__ashrdi3):
- #
- # Checking: Only handle 64bit shifting or more
- #
- cmpb $64, %cl
- jae _Exit
-
- #
- # Handle shifting between 0 and 31 bits
- #
- cmpb $32, %cl
- jae More32
- shrd %cl, %edx, %eax
- shr %cl, %edx
- ret
-
- #
- # Handle shifting of 32-63 bits
- #
-More32:
- movl %edx, %eax
- xor %edx, %edx
- and $31, %cl
- shr %cl, %eax
- ret
-
- #
- # Invalid number (less then 32bits), return 0
- #
-_Exit:
- xor %eax, %eax
- xor %edx, %edx
- ret
diff --git a/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf b/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
index a91c850013..9704b9ea7d 100644
--- a/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
+++ b/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
@@ -39,8 +39,6 @@
Ia32/MathLShiftS64.c | INTEL
Ia32/MathRShiftU64.c | INTEL
- Ia32/MathLShiftS64.S | GCC
- Ia32/MathRShiftU64.S | GCC
Ia32/MathLShiftS64.nasm | GCC
Ia32/MathRShiftU64.nasm | GCC
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v3 11/12] CryptoPkg/IntrinsicLib: Remove .S files for IA32 arch
2019-03-29 7:31 [PATCH v3 11/12] CryptoPkg/IntrinsicLib: Remove .S files for IA32 arch Shenglei Zhang
@ 2019-03-29 8:33 ` Wang, Jian J
0 siblings, 0 replies; 5+ messages in thread
From: Wang, Jian J @ 2019-03-29 8:33 UTC (permalink / raw)
To: Zhang, Shenglei, edk2-devel@lists.01.org; +Cc: Ye, Ting, Gang Wei
Reviewed-by: Jian J Wang <jian.j.wang@intel.com>
> -----Original Message-----
> From: Zhang, Shenglei
> Sent: Friday, March 29, 2019 3:31 PM
> To: edk2-devel@lists.01.org
> Cc: Ye, Ting <ting.ye@intel.com>; Gang Wei <gang.wei@intel.com>; Wang, Jian
> J <jian.j.wang@intel.com>
> Subject: [PATCH v3 11/12] CryptoPkg/IntrinsicLib: Remove .S files for IA32 arch
>
> .nasm file has been added for X86 arch. .S assembly code
> is not required any more.
> https://bugzilla.tianocore.org/show_bug.cgi?id=1594
>
> Cc: Ting Ye <ting.ye@intel.com>
> Cc: Gang Wei <gang.wei@intel.com>
> Cc: Jian Wang <jian.j.wang@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
> ---
> .../Library/IntrinsicLib/Ia32/MathLShiftS64.S | 62 -----------------
> .../Library/IntrinsicLib/Ia32/MathRShiftU64.S | 66 -------------------
> .../Library/IntrinsicLib/IntrinsicLib.inf | 2 -
> 3 files changed, 130 deletions(-)
> delete mode 100644 CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
> delete mode 100644 CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
>
> diff --git a/CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
> b/CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
> deleted file mode 100644
> index 7031a59a71..0000000000
> --- a/CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
> +++ /dev/null
> @@ -1,62 +0,0 @@
> -#------------------------------------------------------------------------------
> -#
> -# Copyright (c) 2014, Intel Corporation. All rights reserved.<BR>
> -# This program and the accompanying materials
> -# are licensed and made available under the terms and conditions of the BSD
> License
> -# which accompanies this distribution. The full text of the license may be found
> at
> -# http://opensource.org/licenses/bsd-license.php.
> -#
> -# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> BASIS,
> -# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> EXPRESS OR IMPLIED.
> -#
> -# Module Name:
> -#
> -# MathLShiftS64.S
> -#
> -# Abstract:
> -#
> -# 64-bit Math Worker Function.
> -# Shifts a 64-bit signed value left by a certain number of bits.
> -#
> -#------------------------------------------------------------------------------
> -
> - .686:
> - .code:
> -
> -ASM_GLOBAL ASM_PFX(__ashldi3)
> -
> -#------------------------------------------------------------------------------
> -#
> -# void __cdecl __ashldi3 (void)
> -#
> -#------------------------------------------------------------------------------
> -ASM_PFX(__ashldi3):
> - #
> - # Handle shifting of 64 or more bits (return 0)
> - #
> - cmpb $64, %cl
> - jae ReturnZero
> -
> - #
> - # Handle shifting of between 0 and 31 bits
> - #
> - cmpb $32, %cl
> - jae More32
> - shld %cl, %eax, %edx
> - shl %cl, %eax
> - ret
> -
> - #
> - # Handle shifting of between 32 and 63 bits
> - #
> -More32:
> - movl %eax, %edx
> - xor %eax, %eax
> - and $31, %cl
> - shl %cl, %edx
> - ret
> -
> -ReturnZero:
> - xor %eax, %eax
> - xor %edx, %edx
> - ret
> diff --git a/CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
> b/CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
> deleted file mode 100644
> index 24142b088e..0000000000
> --- a/CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
> +++ /dev/null
> @@ -1,66 +0,0 @@
> -#------------------------------------------------------------------------------
> -#
> -# Copyright (c) 2014, Intel Corporation. All rights reserved.<BR>
> -# This program and the accompanying materials
> -# are licensed and made available under the terms and conditions of the BSD
> License
> -# which accompanies this distribution. The full text of the license may be found
> at
> -# http://opensource.org/licenses/bsd-license.php.
> -#
> -# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> BASIS,
> -# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> EXPRESS OR IMPLIED.
> -#
> -# Module Name:
> -#
> -# MathRShiftU64.S
> -#
> -# Abstract:
> -#
> -# 64-bit Math Worker Function.
> -# Shifts a 64-bit unsigned value right by a certain number of bits.
> -#
> -#------------------------------------------------------------------------------
> -
> -
> - .686:
> - .code:
> -
> -ASM_GLOBAL ASM_PFX(__ashrdi3)
> -
> -#------------------------------------------------------------------------------
> -#
> -# void __cdecl __ashrdi3 (void)
> -#
> -#------------------------------------------------------------------------------
> -ASM_PFX(__ashrdi3):
> - #
> - # Checking: Only handle 64bit shifting or more
> - #
> - cmpb $64, %cl
> - jae _Exit
> -
> - #
> - # Handle shifting between 0 and 31 bits
> - #
> - cmpb $32, %cl
> - jae More32
> - shrd %cl, %edx, %eax
> - shr %cl, %edx
> - ret
> -
> - #
> - # Handle shifting of 32-63 bits
> - #
> -More32:
> - movl %edx, %eax
> - xor %edx, %edx
> - and $31, %cl
> - shr %cl, %eax
> - ret
> -
> - #
> - # Invalid number (less then 32bits), return 0
> - #
> -_Exit:
> - xor %eax, %eax
> - xor %edx, %edx
> - ret
> diff --git a/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
> b/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
> index a91c850013..9704b9ea7d 100644
> --- a/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
> +++ b/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
> @@ -39,8 +39,6 @@
> Ia32/MathLShiftS64.c | INTEL
> Ia32/MathRShiftU64.c | INTEL
>
> - Ia32/MathLShiftS64.S | GCC
> - Ia32/MathRShiftU64.S | GCC
> Ia32/MathLShiftS64.nasm | GCC
> Ia32/MathRShiftU64.nasm | GCC
>
> --
> 2.18.0.windows.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v3 11/12] CryptoPkg/IntrinsicLib: Remove .S files for IA32 arch
@ 2019-03-29 7:32 Shenglei Zhang
2019-03-29 8:32 ` Wang, Jian J
0 siblings, 1 reply; 5+ messages in thread
From: Shenglei Zhang @ 2019-03-29 7:32 UTC (permalink / raw)
To: edk2-devel; +Cc: Ting Ye, Jian Wang
.nasm file has been added for X86 arch. .S assembly code
is not required any more.
https://bugzilla.tianocore.org/show_bug.cgi?id=1594
Cc: Ting Ye <ting.ye@intel.com>
Cc: Jian Wang <jian.j.wang@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
---
.../Library/IntrinsicLib/Ia32/MathLShiftS64.S | 62 -----------------
.../Library/IntrinsicLib/Ia32/MathRShiftU64.S | 66 -------------------
.../Library/IntrinsicLib/IntrinsicLib.inf | 2 -
3 files changed, 130 deletions(-)
delete mode 100644 CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
delete mode 100644 CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
diff --git a/CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S b/CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
deleted file mode 100644
index 7031a59a71..0000000000
--- a/CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
+++ /dev/null
@@ -1,62 +0,0 @@
-#------------------------------------------------------------------------------
-#
-# Copyright (c) 2014, Intel Corporation. All rights reserved.<BR>
-# This program and the accompanying materials
-# are licensed and made available under the terms and conditions of the BSD License
-# which accompanies this distribution. The full text of the license may be found at
-# http://opensource.org/licenses/bsd-license.php.
-#
-# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
-# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
-#
-# Module Name:
-#
-# MathLShiftS64.S
-#
-# Abstract:
-#
-# 64-bit Math Worker Function.
-# Shifts a 64-bit signed value left by a certain number of bits.
-#
-#------------------------------------------------------------------------------
-
- .686:
- .code:
-
-ASM_GLOBAL ASM_PFX(__ashldi3)
-
-#------------------------------------------------------------------------------
-#
-# void __cdecl __ashldi3 (void)
-#
-#------------------------------------------------------------------------------
-ASM_PFX(__ashldi3):
- #
- # Handle shifting of 64 or more bits (return 0)
- #
- cmpb $64, %cl
- jae ReturnZero
-
- #
- # Handle shifting of between 0 and 31 bits
- #
- cmpb $32, %cl
- jae More32
- shld %cl, %eax, %edx
- shl %cl, %eax
- ret
-
- #
- # Handle shifting of between 32 and 63 bits
- #
-More32:
- movl %eax, %edx
- xor %eax, %eax
- and $31, %cl
- shl %cl, %edx
- ret
-
-ReturnZero:
- xor %eax, %eax
- xor %edx, %edx
- ret
diff --git a/CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S b/CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
deleted file mode 100644
index 24142b088e..0000000000
--- a/CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
+++ /dev/null
@@ -1,66 +0,0 @@
-#------------------------------------------------------------------------------
-#
-# Copyright (c) 2014, Intel Corporation. All rights reserved.<BR>
-# This program and the accompanying materials
-# are licensed and made available under the terms and conditions of the BSD License
-# which accompanies this distribution. The full text of the license may be found at
-# http://opensource.org/licenses/bsd-license.php.
-#
-# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
-# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
-#
-# Module Name:
-#
-# MathRShiftU64.S
-#
-# Abstract:
-#
-# 64-bit Math Worker Function.
-# Shifts a 64-bit unsigned value right by a certain number of bits.
-#
-#------------------------------------------------------------------------------
-
-
- .686:
- .code:
-
-ASM_GLOBAL ASM_PFX(__ashrdi3)
-
-#------------------------------------------------------------------------------
-#
-# void __cdecl __ashrdi3 (void)
-#
-#------------------------------------------------------------------------------
-ASM_PFX(__ashrdi3):
- #
- # Checking: Only handle 64bit shifting or more
- #
- cmpb $64, %cl
- jae _Exit
-
- #
- # Handle shifting between 0 and 31 bits
- #
- cmpb $32, %cl
- jae More32
- shrd %cl, %edx, %eax
- shr %cl, %edx
- ret
-
- #
- # Handle shifting of 32-63 bits
- #
-More32:
- movl %edx, %eax
- xor %edx, %edx
- and $31, %cl
- shr %cl, %eax
- ret
-
- #
- # Invalid number (less then 32bits), return 0
- #
-_Exit:
- xor %eax, %eax
- xor %edx, %edx
- ret
diff --git a/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf b/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
index a91c850013..9704b9ea7d 100644
--- a/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
+++ b/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
@@ -39,8 +39,6 @@
Ia32/MathLShiftS64.c | INTEL
Ia32/MathRShiftU64.c | INTEL
- Ia32/MathLShiftS64.S | GCC
- Ia32/MathRShiftU64.S | GCC
Ia32/MathLShiftS64.nasm | GCC
Ia32/MathRShiftU64.nasm | GCC
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v3 11/12] CryptoPkg/IntrinsicLib: Remove .S files for IA32 arch
2019-03-29 7:32 Shenglei Zhang
@ 2019-03-29 8:32 ` Wang, Jian J
0 siblings, 0 replies; 5+ messages in thread
From: Wang, Jian J @ 2019-03-29 8:32 UTC (permalink / raw)
To: Zhang, Shenglei, edk2-devel@lists.01.org; +Cc: Ye, Ting
Reviewed-by: Jian J Wang <jian.j.wang@intel.com>
> -----Original Message-----
> From: Zhang, Shenglei
> Sent: Friday, March 29, 2019 3:32 PM
> To: edk2-devel@lists.01.org
> Cc: Ye, Ting <ting.ye@intel.com>; Wang, Jian J <jian.j.wang@intel.com>
> Subject: [PATCH v3 11/12] CryptoPkg/IntrinsicLib: Remove .S files for IA32 arch
>
> .nasm file has been added for X86 arch. .S assembly code
> is not required any more.
> https://bugzilla.tianocore.org/show_bug.cgi?id=1594
>
> Cc: Ting Ye <ting.ye@intel.com>
> Cc: Jian Wang <jian.j.wang@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
> ---
> .../Library/IntrinsicLib/Ia32/MathLShiftS64.S | 62 -----------------
> .../Library/IntrinsicLib/Ia32/MathRShiftU64.S | 66 -------------------
> .../Library/IntrinsicLib/IntrinsicLib.inf | 2 -
> 3 files changed, 130 deletions(-)
> delete mode 100644 CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
> delete mode 100644 CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
>
> diff --git a/CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
> b/CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
> deleted file mode 100644
> index 7031a59a71..0000000000
> --- a/CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
> +++ /dev/null
> @@ -1,62 +0,0 @@
> -#------------------------------------------------------------------------------
> -#
> -# Copyright (c) 2014, Intel Corporation. All rights reserved.<BR>
> -# This program and the accompanying materials
> -# are licensed and made available under the terms and conditions of the BSD
> License
> -# which accompanies this distribution. The full text of the license may be found
> at
> -# http://opensource.org/licenses/bsd-license.php.
> -#
> -# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> BASIS,
> -# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> EXPRESS OR IMPLIED.
> -#
> -# Module Name:
> -#
> -# MathLShiftS64.S
> -#
> -# Abstract:
> -#
> -# 64-bit Math Worker Function.
> -# Shifts a 64-bit signed value left by a certain number of bits.
> -#
> -#------------------------------------------------------------------------------
> -
> - .686:
> - .code:
> -
> -ASM_GLOBAL ASM_PFX(__ashldi3)
> -
> -#------------------------------------------------------------------------------
> -#
> -# void __cdecl __ashldi3 (void)
> -#
> -#------------------------------------------------------------------------------
> -ASM_PFX(__ashldi3):
> - #
> - # Handle shifting of 64 or more bits (return 0)
> - #
> - cmpb $64, %cl
> - jae ReturnZero
> -
> - #
> - # Handle shifting of between 0 and 31 bits
> - #
> - cmpb $32, %cl
> - jae More32
> - shld %cl, %eax, %edx
> - shl %cl, %eax
> - ret
> -
> - #
> - # Handle shifting of between 32 and 63 bits
> - #
> -More32:
> - movl %eax, %edx
> - xor %eax, %eax
> - and $31, %cl
> - shl %cl, %edx
> - ret
> -
> -ReturnZero:
> - xor %eax, %eax
> - xor %edx, %edx
> - ret
> diff --git a/CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
> b/CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
> deleted file mode 100644
> index 24142b088e..0000000000
> --- a/CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
> +++ /dev/null
> @@ -1,66 +0,0 @@
> -#------------------------------------------------------------------------------
> -#
> -# Copyright (c) 2014, Intel Corporation. All rights reserved.<BR>
> -# This program and the accompanying materials
> -# are licensed and made available under the terms and conditions of the BSD
> License
> -# which accompanies this distribution. The full text of the license may be found
> at
> -# http://opensource.org/licenses/bsd-license.php.
> -#
> -# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS"
> BASIS,
> -# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER
> EXPRESS OR IMPLIED.
> -#
> -# Module Name:
> -#
> -# MathRShiftU64.S
> -#
> -# Abstract:
> -#
> -# 64-bit Math Worker Function.
> -# Shifts a 64-bit unsigned value right by a certain number of bits.
> -#
> -#------------------------------------------------------------------------------
> -
> -
> - .686:
> - .code:
> -
> -ASM_GLOBAL ASM_PFX(__ashrdi3)
> -
> -#------------------------------------------------------------------------------
> -#
> -# void __cdecl __ashrdi3 (void)
> -#
> -#------------------------------------------------------------------------------
> -ASM_PFX(__ashrdi3):
> - #
> - # Checking: Only handle 64bit shifting or more
> - #
> - cmpb $64, %cl
> - jae _Exit
> -
> - #
> - # Handle shifting between 0 and 31 bits
> - #
> - cmpb $32, %cl
> - jae More32
> - shrd %cl, %edx, %eax
> - shr %cl, %edx
> - ret
> -
> - #
> - # Handle shifting of 32-63 bits
> - #
> -More32:
> - movl %edx, %eax
> - xor %edx, %edx
> - and $31, %cl
> - shr %cl, %eax
> - ret
> -
> - #
> - # Invalid number (less then 32bits), return 0
> - #
> -_Exit:
> - xor %eax, %eax
> - xor %edx, %edx
> - ret
> diff --git a/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
> b/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
> index a91c850013..9704b9ea7d 100644
> --- a/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
> +++ b/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
> @@ -39,8 +39,6 @@
> Ia32/MathLShiftS64.c | INTEL
> Ia32/MathRShiftU64.c | INTEL
>
> - Ia32/MathLShiftS64.S | GCC
> - Ia32/MathRShiftU64.S | GCC
> Ia32/MathLShiftS64.nasm | GCC
> Ia32/MathRShiftU64.nasm | GCC
>
> --
> 2.18.0.windows.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v3 00/12] Remove .S files for IA32 and X64 arch in MdePkg and UefiCpuPkg
@ 2019-03-29 7:28 Shenglei Zhang
2019-03-29 7:28 ` [PATCH v3 11/12] CryptoPkg/IntrinsicLib: Remove .S files for IA32 arch Shenglei Zhang
0 siblings, 1 reply; 5+ messages in thread
From: Shenglei Zhang @ 2019-03-29 7:28 UTC (permalink / raw)
To: edk2-devel; +Cc: Michael D Kinney, Liming Gao, Eric Dong, Ray Ni
.nasm file has been added for X86 arch. .S assembly code
is not required any more.
https://bugzilla.tianocore.org/show_bug.cgi?id=1594
v2: Remove some description in 04/10.
v3: Add 11/12 and 12/12.
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Shenglei Zhang (12):
UefiCpuPkg/SmmCpuFeaturesLib: Remove .S files for IA32 and X64 arch
UefiCpuPkg/BaseUefiCpuLib: Remove .S files for IA32 and X64 arch
UefiCpuPkg/CpuExceptionHandlerLib:Remove.S files for IA32 and X64 arch
MdePkg/BaseCpuLib: Remove .S files for IA32 and X64 arch
MdePkg/BaseLib: Remove .S files for IA32 and X64 arch
MdePkg/BaseMemoryLibMmx: Remove .S files for IA32 and X64 arch
MdePkg/BaseMemoryLibOptDxe: Remove .S files for IA32 and X64 arch
MdePkg/BaseMemoryLibOptPei: Remove .S files for IA32 and X64 arch
MdePkg/BaseMemoryLibRepStr: Remove .S files for IA32 and X64 arch
MdePkg/BaseMemoryLibSse2: Remove .S files for IA32 and X64 arch
CryptoPkg/IntrinsicLib: Remove .S files for IA32 arch
SourceLevelDebugPkg/DebugAgentCommon: Remove .S files
.../Library/IntrinsicLib/Ia32/MathLShiftS64.S | 62 --
.../Library/IntrinsicLib/Ia32/MathRShiftU64.S | 66 --
.../Library/IntrinsicLib/IntrinsicLib.inf | 2 -
MdePkg/Library/BaseCpuLib/BaseCpuLib.inf | 4 -
MdePkg/Library/BaseCpuLib/X64/CpuFlushTlb.S | 35 -
MdePkg/Library/BaseCpuLib/X64/CpuSleep.S | 34 -
MdePkg/Library/BaseLib/BaseLib.inf | 38 -
MdePkg/Library/BaseLib/Ia32/ARShiftU64.S | 43 --
MdePkg/Library/BaseLib/Ia32/CpuId.S | 63 --
MdePkg/Library/BaseLib/Ia32/CpuIdEx.S | 67 --
MdePkg/Library/BaseLib/Ia32/DisableCache.S | 39 -
MdePkg/Library/BaseLib/Ia32/DisablePaging32.S | 52 --
MdePkg/Library/BaseLib/Ia32/DivU64x32.S | 41 --
.../Library/BaseLib/Ia32/DivU64x32Remainder.S | 46 --
.../Library/BaseLib/Ia32/DivU64x64Remainder.S | 89 ---
MdePkg/Library/BaseLib/Ia32/EnableCache.S | 39 -
.../BaseLib/Ia32/EnableDisableInterrupts.S | 36 -
MdePkg/Library/BaseLib/Ia32/EnablePaging32.S | 52 --
MdePkg/Library/BaseLib/Ia32/EnablePaging64.S | 63 --
.../BaseLib/Ia32/InternalSwitchStack.S | 48 --
MdePkg/Library/BaseLib/Ia32/LRotU64.S | 48 --
MdePkg/Library/BaseLib/Ia32/LShiftU64.S | 43 --
MdePkg/Library/BaseLib/Ia32/LongJump.S | 41 --
MdePkg/Library/BaseLib/Ia32/ModU64x32.S | 40 --
MdePkg/Library/BaseLib/Ia32/Monitor.S | 40 --
MdePkg/Library/BaseLib/Ia32/MultU64x32.S | 41 --
MdePkg/Library/BaseLib/Ia32/MultU64x64.S | 44 --
MdePkg/Library/BaseLib/Ia32/Mwait.S | 38 -
MdePkg/Library/BaseLib/Ia32/RRotU64.S | 48 --
MdePkg/Library/BaseLib/Ia32/RShiftU64.S | 46 --
MdePkg/Library/BaseLib/Ia32/RdRand.S | 80 ---
MdePkg/Library/BaseLib/Ia32/SetJump.S | 44 --
MdePkg/Library/BaseLib/Ia32/SwapBytes64.S | 38 -
MdePkg/Library/BaseLib/Ia32/Thunk16.S | 222 ------
MdePkg/Library/BaseLib/X64/CpuId.S | 60 --
MdePkg/Library/BaseLib/X64/CpuIdEx.S | 62 --
MdePkg/Library/BaseLib/X64/DisableCache.S | 39 -
MdePkg/Library/BaseLib/X64/DisablePaging64.S | 82 ---
MdePkg/Library/BaseLib/X64/EnableCache.S | 39 -
.../BaseLib/X64/EnableDisableInterrupts.S | 36 -
MdePkg/Library/BaseLib/X64/LongJump.S | 54 --
MdePkg/Library/BaseLib/X64/RdRand.S | 72 --
MdePkg/Library/BaseLib/X64/SetJump.S | 53 --
MdePkg/Library/BaseLib/X64/SwitchStack.S | 52 --
MdePkg/Library/BaseLib/X64/Thunk16.S | 334 ---------
.../BaseMemoryLibMmx/BaseMemoryLibMmx.inf | 22 -
.../BaseMemoryLibMmx/Ia32/CompareMem.S | 55 --
.../Library/BaseMemoryLibMmx/Ia32/CopyMem.S | 86 ---
.../Library/BaseMemoryLibMmx/Ia32/ScanMem16.S | 52 --
.../Library/BaseMemoryLibMmx/Ia32/ScanMem32.S | 52 --
.../Library/BaseMemoryLibMmx/Ia32/ScanMem64.S | 61 --
.../Library/BaseMemoryLibMmx/Ia32/ScanMem8.S | 52 --
MdePkg/Library/BaseMemoryLibMmx/Ia32/SetMem.S | 66 --
.../Library/BaseMemoryLibMmx/Ia32/SetMem16.S | 59 --
.../Library/BaseMemoryLibMmx/Ia32/SetMem32.S | 52 --
.../Library/BaseMemoryLibMmx/Ia32/SetMem64.S | 43 --
.../Library/BaseMemoryLibMmx/Ia32/ZeroMem.S | 54 --
.../Library/BaseMemoryLibMmx/X64/CompareMem.S | 59 --
MdePkg/Library/BaseMemoryLibMmx/X64/CopyMem.S | 74 --
.../Library/BaseMemoryLibMmx/X64/ScanMem16.S | 56 --
.../Library/BaseMemoryLibMmx/X64/ScanMem32.S | 56 --
.../Library/BaseMemoryLibMmx/X64/ScanMem64.S | 55 --
.../Library/BaseMemoryLibMmx/X64/ScanMem8.S | 56 --
MdePkg/Library/BaseMemoryLibMmx/X64/SetMem.S | 61 --
.../Library/BaseMemoryLibMmx/X64/SetMem16.S | 60 --
.../Library/BaseMemoryLibMmx/X64/SetMem32.S | 55 --
.../Library/BaseMemoryLibMmx/X64/SetMem64.S | 47 --
MdePkg/Library/BaseMemoryLibMmx/X64/ZeroMem.S | 57 --
.../BaseMemoryLibOptDxe.inf | 22 -
.../BaseMemoryLibOptDxe/Ia32/CompareMem.S | 55 --
.../BaseMemoryLibOptDxe/Ia32/CopyMem.S | 85 ---
.../BaseMemoryLibOptDxe/Ia32/ScanMem16.S | 52 --
.../BaseMemoryLibOptDxe/Ia32/ScanMem32.S | 52 --
.../BaseMemoryLibOptDxe/Ia32/ScanMem64.S | 61 --
.../BaseMemoryLibOptDxe/Ia32/ScanMem8.S | 52 --
.../Library/BaseMemoryLibOptDxe/Ia32/SetMem.S | 50 --
.../BaseMemoryLibOptDxe/Ia32/SetMem16.S | 43 --
.../BaseMemoryLibOptDxe/Ia32/SetMem32.S | 43 --
.../BaseMemoryLibOptDxe/Ia32/SetMem64.S | 46 --
.../BaseMemoryLibOptDxe/Ia32/ZeroMem.S | 49 --
.../BaseMemoryLibOptDxe/X64/CompareMem.S | 59 --
.../Library/BaseMemoryLibOptDxe/X64/CopyMem.S | 82 ---
.../BaseMemoryLibOptDxe/X64/ScanMem16.S | 56 --
.../BaseMemoryLibOptDxe/X64/ScanMem32.S | 56 --
.../BaseMemoryLibOptDxe/X64/ScanMem64.S | 55 --
.../BaseMemoryLibOptDxe/X64/ScanMem8.S | 56 --
.../Library/BaseMemoryLibOptDxe/X64/SetMem.S | 57 --
.../BaseMemoryLibOptDxe/X64/SetMem16.S | 47 --
.../BaseMemoryLibOptDxe/X64/SetMem32.S | 47 --
.../BaseMemoryLibOptDxe/X64/SetMem64.S | 46 --
.../Library/BaseMemoryLibOptDxe/X64/ZeroMem.S | 51 --
.../BaseMemoryLibOptPei.inf | 22 -
.../BaseMemoryLibOptPei/Ia32/CompareMem.S | 55 --
.../BaseMemoryLibOptPei/Ia32/CopyMem.S | 62 --
.../BaseMemoryLibOptPei/Ia32/ScanMem16.S | 52 --
.../BaseMemoryLibOptPei/Ia32/ScanMem32.S | 52 --
.../BaseMemoryLibOptPei/Ia32/ScanMem64.S | 61 --
.../BaseMemoryLibOptPei/Ia32/ScanMem8.S | 52 --
.../Library/BaseMemoryLibOptPei/Ia32/SetMem.S | 50 --
.../BaseMemoryLibOptPei/Ia32/SetMem16.S | 43 --
.../BaseMemoryLibOptPei/Ia32/SetMem32.S | 43 --
.../BaseMemoryLibOptPei/Ia32/SetMem64.S | 46 --
.../BaseMemoryLibOptPei/Ia32/ZeroMem.S | 49 --
.../BaseMemoryLibOptPei/X64/CompareMem.S | 59 --
.../Library/BaseMemoryLibOptPei/X64/CopyMem.S | 66 --
.../BaseMemoryLibOptPei/X64/ScanMem16.S | 56 --
.../BaseMemoryLibOptPei/X64/ScanMem32.S | 56 --
.../BaseMemoryLibOptPei/X64/ScanMem64.S | 56 --
.../BaseMemoryLibOptPei/X64/ScanMem8.S | 56 --
.../Library/BaseMemoryLibOptPei/X64/SetMem.S | 47 --
.../BaseMemoryLibOptPei/X64/SetMem16.S | 47 --
.../BaseMemoryLibOptPei/X64/SetMem32.S | 47 --
.../BaseMemoryLibOptPei/X64/SetMem64.S | 46 --
.../Library/BaseMemoryLibOptPei/X64/ZeroMem.S | 50 --
.../BaseMemoryLibRepStr.inf | 22 -
.../BaseMemoryLibRepStr/Ia32/CompareMem.S | 55 --
.../BaseMemoryLibRepStr/Ia32/CopyMem.S | 65 --
.../BaseMemoryLibRepStr/Ia32/ScanMem16.S | 54 --
.../BaseMemoryLibRepStr/Ia32/ScanMem32.S | 54 --
.../BaseMemoryLibRepStr/Ia32/ScanMem64.S | 63 --
.../BaseMemoryLibRepStr/Ia32/ScanMem8.S | 54 --
.../Library/BaseMemoryLibRepStr/Ia32/SetMem.S | 46 --
.../BaseMemoryLibRepStr/Ia32/SetMem16.S | 43 --
.../BaseMemoryLibRepStr/Ia32/SetMem32.S | 43 --
.../BaseMemoryLibRepStr/Ia32/SetMem64.S | 46 --
.../BaseMemoryLibRepStr/Ia32/ZeroMem.S | 49 --
.../BaseMemoryLibRepStr/X64/CompareMem.S | 59 --
.../Library/BaseMemoryLibRepStr/X64/CopyMem.S | 66 --
.../BaseMemoryLibRepStr/X64/ScanMem16.S | 56 --
.../BaseMemoryLibRepStr/X64/ScanMem32.S | 56 --
.../BaseMemoryLibRepStr/X64/ScanMem64.S | 56 --
.../BaseMemoryLibRepStr/X64/ScanMem8.S | 56 --
.../Library/BaseMemoryLibRepStr/X64/SetMem.S | 47 --
.../BaseMemoryLibRepStr/X64/SetMem16.S | 47 --
.../BaseMemoryLibRepStr/X64/SetMem32.S | 47 --
.../BaseMemoryLibRepStr/X64/SetMem64.S | 46 --
.../Library/BaseMemoryLibRepStr/X64/ZeroMem.S | 50 --
.../BaseMemoryLibSse2/BaseMemoryLibSse2.inf | 22 -
.../BaseMemoryLibSse2/Ia32/CompareMem.S | 55 --
.../Library/BaseMemoryLibSse2/Ia32/CopyMem.S | 85 ---
.../BaseMemoryLibSse2/Ia32/ScanMem16.S | 52 --
.../BaseMemoryLibSse2/Ia32/ScanMem32.S | 52 --
.../BaseMemoryLibSse2/Ia32/ScanMem64.S | 61 --
.../Library/BaseMemoryLibSse2/Ia32/ScanMem8.S | 52 --
.../Library/BaseMemoryLibSse2/Ia32/SetMem.S | 76 --
.../Library/BaseMemoryLibSse2/Ia32/SetMem16.S | 69 --
.../Library/BaseMemoryLibSse2/Ia32/SetMem32.S | 68 --
.../Library/BaseMemoryLibSse2/Ia32/SetMem64.S | 58 --
.../Library/BaseMemoryLibSse2/Ia32/ZeroMem.S | 65 --
.../BaseMemoryLibSse2/X64/CompareMem.S | 59 --
.../Library/BaseMemoryLibSse2/X64/CopyMem.S | 83 ---
.../Library/BaseMemoryLibSse2/X64/ScanMem16.S | 56 --
.../Library/BaseMemoryLibSse2/X64/ScanMem32.S | 56 --
.../Library/BaseMemoryLibSse2/X64/ScanMem64.S | 56 --
.../Library/BaseMemoryLibSse2/X64/ScanMem8.S | 56 --
MdePkg/Library/BaseMemoryLibSse2/X64/SetMem.S | 72 --
.../Library/BaseMemoryLibSse2/X64/SetMem16.S | 70 --
.../Library/BaseMemoryLibSse2/X64/SetMem32.S | 69 --
.../Library/BaseMemoryLibSse2/X64/SetMem64.S | 60 --
.../Library/BaseMemoryLibSse2/X64/ZeroMem.S | 65 --
.../DebugAgentCommon/Ia32/AsmFuncs.S | 415 -----------
.../DebugAgentCommon/X64/AsmFuncs.S | 431 -----------
.../Library/DebugAgent/DxeDebugAgentLib.inf | 2 -
.../Library/BaseUefiCpuLib/BaseUefiCpuLib.inf | 2 -
.../BaseUefiCpuLib/Ia32/InitializeFpu.S | 73 --
.../BaseUefiCpuLib/X64/InitializeFpu.S | 57 --
.../DxeCpuExceptionHandlerLib.inf | 2 -
.../Ia32/ExceptionHandlerAsm.S | 667 ------------------
.../PeiCpuExceptionHandlerLib.inf | 2 -
.../SecPeiCpuExceptionHandlerLib.inf | 2 -
.../SmmCpuExceptionHandlerLib.inf | 2 -
.../X64/ExceptionHandlerAsm.S | 434 ------------
.../Library/SmmCpuFeaturesLib/Ia32/SmiEntry.S | 278 --------
.../SmmCpuFeaturesLib/Ia32/SmiException.S | 174 -----
.../SmmCpuFeaturesLibStm.inf | 6 -
.../Library/SmmCpuFeaturesLib/X64/SmiEntry.S | 282 --------
.../SmmCpuFeaturesLib/X64/SmiException.S | 178 -----
177 files changed, 11904 deletions(-)
delete mode 100644 CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
delete mode 100644 CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
delete mode 100644 MdePkg/Library/BaseCpuLib/X64/CpuFlushTlb.S
delete mode 100644 MdePkg/Library/BaseCpuLib/X64/CpuSleep.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/ARShiftU64.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/CpuId.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/CpuIdEx.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/DisableCache.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/DisablePaging32.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/DivU64x32.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/DivU64x32Remainder.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/DivU64x64Remainder.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/EnableCache.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/EnableDisableInterrupts.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/EnablePaging32.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/EnablePaging64.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/InternalSwitchStack.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/LRotU64.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/LShiftU64.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/LongJump.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/ModU64x32.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/Monitor.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/MultU64x32.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/MultU64x64.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/Mwait.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/RRotU64.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/RShiftU64.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/RdRand.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/SetJump.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/SwapBytes64.S
delete mode 100644 MdePkg/Library/BaseLib/Ia32/Thunk16.S
delete mode 100644 MdePkg/Library/BaseLib/X64/CpuId.S
delete mode 100644 MdePkg/Library/BaseLib/X64/CpuIdEx.S
delete mode 100644 MdePkg/Library/BaseLib/X64/DisableCache.S
delete mode 100644 MdePkg/Library/BaseLib/X64/DisablePaging64.S
delete mode 100644 MdePkg/Library/BaseLib/X64/EnableCache.S
delete mode 100644 MdePkg/Library/BaseLib/X64/EnableDisableInterrupts.S
delete mode 100644 MdePkg/Library/BaseLib/X64/LongJump.S
delete mode 100644 MdePkg/Library/BaseLib/X64/RdRand.S
delete mode 100644 MdePkg/Library/BaseLib/X64/SetJump.S
delete mode 100644 MdePkg/Library/BaseLib/X64/SwitchStack.S
delete mode 100644 MdePkg/Library/BaseLib/X64/Thunk16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/Ia32/CompareMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/Ia32/CopyMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/Ia32/ScanMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/Ia32/ScanMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/Ia32/ScanMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/Ia32/ScanMem8.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/Ia32/SetMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/Ia32/SetMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/Ia32/SetMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/Ia32/SetMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/Ia32/ZeroMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/X64/CompareMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/X64/CopyMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/X64/ScanMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/X64/ScanMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/X64/ScanMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/X64/ScanMem8.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/X64/SetMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/X64/SetMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/X64/SetMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/X64/SetMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibMmx/X64/ZeroMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/Ia32/CompareMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/Ia32/CopyMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/Ia32/ScanMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/Ia32/ScanMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/Ia32/ScanMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/Ia32/ScanMem8.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/Ia32/SetMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/Ia32/SetMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/Ia32/SetMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/Ia32/SetMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/Ia32/ZeroMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/X64/CompareMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/X64/CopyMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/X64/ScanMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/X64/ScanMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/X64/ScanMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/X64/ScanMem8.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/X64/SetMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/X64/SetMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/X64/SetMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/X64/SetMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptDxe/X64/ZeroMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/Ia32/CompareMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/Ia32/CopyMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/Ia32/ScanMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/Ia32/ScanMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/Ia32/ScanMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/Ia32/ScanMem8.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/Ia32/SetMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/Ia32/SetMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/Ia32/SetMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/Ia32/SetMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/Ia32/ZeroMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/X64/CompareMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/X64/CopyMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/X64/ScanMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/X64/ScanMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/X64/ScanMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/X64/ScanMem8.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/X64/SetMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/X64/SetMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/X64/SetMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/X64/SetMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibOptPei/X64/ZeroMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/Ia32/CompareMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/Ia32/CopyMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/Ia32/ScanMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/Ia32/ScanMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/Ia32/ScanMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/Ia32/ScanMem8.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/Ia32/SetMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/Ia32/SetMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/Ia32/SetMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/Ia32/SetMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/Ia32/ZeroMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/X64/CompareMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/X64/CopyMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/X64/ScanMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/X64/ScanMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/X64/ScanMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/X64/ScanMem8.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/X64/SetMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/X64/SetMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/X64/SetMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/X64/SetMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibRepStr/X64/ZeroMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/Ia32/CompareMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/Ia32/CopyMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/Ia32/ScanMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/Ia32/ScanMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/Ia32/ScanMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/Ia32/ScanMem8.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/Ia32/SetMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/Ia32/SetMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/Ia32/SetMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/Ia32/SetMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/Ia32/ZeroMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/X64/CompareMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/X64/CopyMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/X64/ScanMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/X64/ScanMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/X64/ScanMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/X64/ScanMem8.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/X64/SetMem.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/X64/SetMem16.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/X64/SetMem32.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/X64/SetMem64.S
delete mode 100644 MdePkg/Library/BaseMemoryLibSse2/X64/ZeroMem.S
delete mode 100644 SourceLevelDebugPkg/Library/DebugAgent/DebugAgentCommon/Ia32/AsmFuncs.S
delete mode 100644 SourceLevelDebugPkg/Library/DebugAgent/DebugAgentCommon/X64/AsmFuncs.S
delete mode 100644 UefiCpuPkg/Library/BaseUefiCpuLib/Ia32/InitializeFpu.S
delete mode 100644 UefiCpuPkg/Library/BaseUefiCpuLib/X64/InitializeFpu.S
delete mode 100644 UefiCpuPkg/Library/CpuExceptionHandlerLib/Ia32/ExceptionHandlerAsm.S
delete mode 100644 UefiCpuPkg/Library/CpuExceptionHandlerLib/X64/ExceptionHandlerAsm.S
delete mode 100644 UefiCpuPkg/Library/SmmCpuFeaturesLib/Ia32/SmiEntry.S
delete mode 100644 UefiCpuPkg/Library/SmmCpuFeaturesLib/Ia32/SmiException.S
delete mode 100644 UefiCpuPkg/Library/SmmCpuFeaturesLib/X64/SmiEntry.S
delete mode 100644 UefiCpuPkg/Library/SmmCpuFeaturesLib/X64/SmiException.S
--
2.18.0.windows.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v3 11/12] CryptoPkg/IntrinsicLib: Remove .S files for IA32 arch
2019-03-29 7:28 [PATCH v3 00/12] Remove .S files for IA32 and X64 arch in MdePkg and UefiCpuPkg Shenglei Zhang
@ 2019-03-29 7:28 ` Shenglei Zhang
0 siblings, 0 replies; 5+ messages in thread
From: Shenglei Zhang @ 2019-03-29 7:28 UTC (permalink / raw)
To: edk2-devel; +Cc: Ting Ye, Gang Wei, Jian Wang
.nasm file has been added for X86 arch. .S assembly code
is not required any more.
https://bugzilla.tianocore.org/show_bug.cgi?id=1594
Cc: Ting Ye <ting.ye@intel.com>
Cc: Gang Wei <gang.wei@intel.com>
Cc: Jian Wang <jian.j.wang@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
---
.../Library/IntrinsicLib/Ia32/MathLShiftS64.S | 62 -----------------
.../Library/IntrinsicLib/Ia32/MathRShiftU64.S | 66 -------------------
.../Library/IntrinsicLib/IntrinsicLib.inf | 2 -
3 files changed, 130 deletions(-)
delete mode 100644 CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
delete mode 100644 CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
diff --git a/CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S b/CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
deleted file mode 100644
index 7031a59a71..0000000000
--- a/CryptoPkg/Library/IntrinsicLib/Ia32/MathLShiftS64.S
+++ /dev/null
@@ -1,62 +0,0 @@
-#------------------------------------------------------------------------------
-#
-# Copyright (c) 2014, Intel Corporation. All rights reserved.<BR>
-# This program and the accompanying materials
-# are licensed and made available under the terms and conditions of the BSD License
-# which accompanies this distribution. The full text of the license may be found at
-# http://opensource.org/licenses/bsd-license.php.
-#
-# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
-# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
-#
-# Module Name:
-#
-# MathLShiftS64.S
-#
-# Abstract:
-#
-# 64-bit Math Worker Function.
-# Shifts a 64-bit signed value left by a certain number of bits.
-#
-#------------------------------------------------------------------------------
-
- .686:
- .code:
-
-ASM_GLOBAL ASM_PFX(__ashldi3)
-
-#------------------------------------------------------------------------------
-#
-# void __cdecl __ashldi3 (void)
-#
-#------------------------------------------------------------------------------
-ASM_PFX(__ashldi3):
- #
- # Handle shifting of 64 or more bits (return 0)
- #
- cmpb $64, %cl
- jae ReturnZero
-
- #
- # Handle shifting of between 0 and 31 bits
- #
- cmpb $32, %cl
- jae More32
- shld %cl, %eax, %edx
- shl %cl, %eax
- ret
-
- #
- # Handle shifting of between 32 and 63 bits
- #
-More32:
- movl %eax, %edx
- xor %eax, %eax
- and $31, %cl
- shl %cl, %edx
- ret
-
-ReturnZero:
- xor %eax, %eax
- xor %edx, %edx
- ret
diff --git a/CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S b/CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
deleted file mode 100644
index 24142b088e..0000000000
--- a/CryptoPkg/Library/IntrinsicLib/Ia32/MathRShiftU64.S
+++ /dev/null
@@ -1,66 +0,0 @@
-#------------------------------------------------------------------------------
-#
-# Copyright (c) 2014, Intel Corporation. All rights reserved.<BR>
-# This program and the accompanying materials
-# are licensed and made available under the terms and conditions of the BSD License
-# which accompanies this distribution. The full text of the license may be found at
-# http://opensource.org/licenses/bsd-license.php.
-#
-# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
-# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
-#
-# Module Name:
-#
-# MathRShiftU64.S
-#
-# Abstract:
-#
-# 64-bit Math Worker Function.
-# Shifts a 64-bit unsigned value right by a certain number of bits.
-#
-#------------------------------------------------------------------------------
-
-
- .686:
- .code:
-
-ASM_GLOBAL ASM_PFX(__ashrdi3)
-
-#------------------------------------------------------------------------------
-#
-# void __cdecl __ashrdi3 (void)
-#
-#------------------------------------------------------------------------------
-ASM_PFX(__ashrdi3):
- #
- # Checking: Only handle 64bit shifting or more
- #
- cmpb $64, %cl
- jae _Exit
-
- #
- # Handle shifting between 0 and 31 bits
- #
- cmpb $32, %cl
- jae More32
- shrd %cl, %edx, %eax
- shr %cl, %edx
- ret
-
- #
- # Handle shifting of 32-63 bits
- #
-More32:
- movl %edx, %eax
- xor %edx, %edx
- and $31, %cl
- shr %cl, %eax
- ret
-
- #
- # Invalid number (less then 32bits), return 0
- #
-_Exit:
- xor %eax, %eax
- xor %edx, %edx
- ret
diff --git a/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf b/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
index a91c850013..9704b9ea7d 100644
--- a/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
+++ b/CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf
@@ -39,8 +39,6 @@
Ia32/MathLShiftS64.c | INTEL
Ia32/MathRShiftU64.c | INTEL
- Ia32/MathLShiftS64.S | GCC
- Ia32/MathRShiftU64.S | GCC
Ia32/MathLShiftS64.nasm | GCC
Ia32/MathRShiftU64.nasm | GCC
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-03-29 8:33 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-03-29 7:31 [PATCH v3 11/12] CryptoPkg/IntrinsicLib: Remove .S files for IA32 arch Shenglei Zhang
2019-03-29 8:33 ` Wang, Jian J
-- strict thread matches above, loose matches on Subject: below --
2019-03-29 7:32 Shenglei Zhang
2019-03-29 8:32 ` Wang, Jian J
2019-03-29 7:28 [PATCH v3 00/12] Remove .S files for IA32 and X64 arch in MdePkg and UefiCpuPkg Shenglei Zhang
2019-03-29 7:28 ` [PATCH v3 11/12] CryptoPkg/IntrinsicLib: Remove .S files for IA32 arch Shenglei Zhang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox