Re: [edk2-devel] 回复: [edk2-devel] 回复: [edk2-devel] [PATCH] Support GLOBAL_REMOVE_IF_UNREFERENCED in GCC5/11

2023-05-04 Thread JoeX Lu
OK, thanks for your comment. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#104090): https://edk2.groups.io/g/devel/message/104090 Mute This Topic: https://groups.io/mt/98698470/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: h

Re: [edk2-devel] 回复: [edk2-devel] [PATCH] Support GLOBAL_REMOVE_IF_UNREFERENCED in GCC5/11

2023-05-04 Thread JoeX Lu
Hi Liming, For question1: We might declaim some variables/functions only used in DEBUG mode and those are necessary variables/functions for debugging but will trigger warning message in RELEASE build. For question2: Yes -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to

Re: [edk2-devel] [PATCH] Support GLOBAL_REMOVE_IF_UNREFERENCED in GCC5/11

2023-04-27 Thread JoeX Lu
Hi Pedro, Thanks for your comment. I hope this attribute can silent Wunused in GCC build. Because it will be nice to declaim GLOBAL_REMOVE_IF_UNREFERENCED before unusd variable than #ifdef example1: #ifdef (UNUSED_CONDITION) static Boolean samplvariable; // which will be triggered the Wunused

[edk2-devel] [PATCH] Support GLOBAL_REMOVE_IF_UNREFERENCED in GCC5/11

2023-04-27 Thread JoeX Lu
CC: Michael D Kinney CC: Liming Gao CC: Zhiguang Liu Signed-off-by: JoeX Lu --- MdePkg/Include/Base.h | 4 1 file changed, 4 insertions(+) diff --git a/MdePkg/Include/Base.h b/MdePkg/Include/Base.h index 6597e441a6..951fce43ee 100644 --- a/MdePkg/Include/Base.h +++ b/MdePkg/Include

Re: [edk2-devel] [PATCH] Release build in GCC5 has warning message Wreturn-local-addr in HashPeiLib

2023-04-27 Thread JoeX Lu
-ChunX > Sent: Thursday, April 27, 2023 2:59 PM > To: devel@edk2.groups.io > Cc: Lu, Pen-ChunX ; Yao, Jiewen > ; Wang, Jian J > Subject: [PATCH] Release build in GCC5 has warning message > Wreturn-local- addr in HashPeiLib > > CC: Jiewen Yao > CC: Jian

[edk2-devel] [PATCH] Release build in GCC5 has warning message Wreturn-local-addr in HashPeiLib

2023-04-27 Thread JoeX Lu
CC: Jiewen Yao CC: Jian J Wang Signed-off-by: JoeX Lu --- .../HashLibBaseCryptoRouterPei.c| 17 + 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/SecurityPkg/Library/HashLibBaseCryptoRouter/HashLibBaseCryptoRouterPei.c b/SecurityPkg/Library

[edk2-devel] [PATCH] Release build in GCC5 has warning message Wreturn-local-addr in HiiLib

2023-04-26 Thread JoeX Lu
CC: Jian J Wang CC: Liming Gao CC: Dandan Bi CC: Eric Dong CC: devel@edk2.groups.io Signed-off-by: JoeX Lu --- MdeModulePkg/Library/UefiHiiLib/HiiLib.c | 20 +--- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/MdeModulePkg/Library/UefiHiiLib/HiiLib.c b

Re: [edk2-devel] [PATCH] Fix GCC5 Release build warning [-Wreturn-local-addr]

2023-04-26 Thread JoeX Lu
packages. Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io 代表 JoeX Lu > 发送时间: 2023年4月25日 14:40 > 收件人: devel@edk2.groups.io > 抄送: JoeX Lu ; Jian J Wang > ; Liming Gao ; Dandan > Bi ; Eric Dong ; Jiewen Yao > > 主题: [edk2-devel] [PATCH] Fix GCC5 Release build

Recall: [edk2-devel] [PATCH] Fix GCC5 Release build warning [-Wreturn-local-addr]

2023-04-26 Thread JoeX Lu
Lu, Pen-ChunX would like to recall the message, "[edk2-devel] [PATCH] Fix GCC5 Release build warning [-Wreturn-local-addr]". -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#103685): https://edk2.groups.io/g/devel/message/103685 Mute This T

Re: [edk2-devel] [PATCH] Fix GCC5 Release build warning [-Wreturn-local-addr]

2023-04-26 Thread JoeX Lu
is warning message? And, please separate the patch for the different packages. Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io 代表 JoeX Lu > 发送时间: 2023年4月25日 14:40 > 收件人: devel@edk2.groups.io > 抄送: JoeX Lu ; Jian J Wang > ; Liming Gao ; Dandan > Bi ; Eric Dong ; J

[edk2-devel] [PATCH] Fix GCC5 Release build warning [-Wreturn-local-addr]

2023-04-26 Thread JoeX Lu
Cc: Jian J Wang Cc: Liming Gao Cc: Dandan Bi Cc: Eric Dong Cc: devel@edk2.groups.io Cc: Jiewen Yao Signed-off-by: JoeX Lu --- MdeModulePkg/Library/UefiHiiLib/HiiLib.c | 20 --- .../HashLibBaseCryptoRouterPei.c | 19 ++ 2 files changed, 28