Hello Oliver, Thanks for testing the patch and pointing out the problem. Yes, the patch does have some missing '-c' flags for gcc versions other than 4.9. That will be fixed in the next patch, probably later today.
The patch is not intended to change the -Wno-unused-but-set-variable use. -Wunused-but-set-variable is initially set by -Wall and works well for non-release builds. But in the release build where ASSERTS are removed, some supporting locals remain and trigger the warning. So this warning is disabled for release builds. AARCH64 builds are working well using the Windows hosted gcc compiler, though I am building only and not boot testing. Only AppPkg fails to build, and you explained that problem: http://sourceforge.net/p/edk2/mailman/message/32988023/ The Windows hosted gcc compiler is not doing as well for ARM. I need to do more work there. Thanks, Scott -----Original Message----- From: Olivier Martin [mailto:[email protected]] Sent: Monday, November 10, 2014 04:11 AM To: [email protected] Subject: Re: [edk2] Status of gcc link time optimization for EDK2 use Interesting work, I also tried earlier this year, but I did not manage to get something working quickly enough. I had to postpone this task. I had a quick look at the patch, I guess these changes: DEFINE GCC49_ARM_DEBUG_FLAGS = -O0 DEFINE GCC49_ARM_RELEASE_FLAGS = -Wno-unused-but-set-variable DEFINE GCC49_AARCH64_DEBUG_FLAGS = -O0 DEFINE GCC49_AARCH64_RELEASE_FLAGS = -Wno-unused-but-set-variable ... is to fix the ShellPkg build error I raised 10 days ago. Your patch looks to work fine in our CI with AArch64. But interestingly I have this build error with ARM GCC48: "/arm/tools/arm/gcc-arm-none-eabi/4_8-2014q2/bin/arm-none-eabi-gcc" -mthumb -march=armv7-a -g -fshort-wchar -fno-strict-aliasing -Wall -Werror -Wno-array-bounds -ffunction-sections -fdata-sections -include AutoGen.h -DSTRING_ARRAY_NAME=BaseMemoryLibStmStrings -g -Os -fshort-wchar -fno-strict-aliasing -Wall -Werror -Wno-array-bounds -include AutoGen.h -mword-relocations -mlittle-endian -mabi=aapcs -mapcs -fno-short-enums -save-temps -fsigned-char -ffunction-sections -fdata-sections -fomit-frame-pointer -Wno-address -mthumb -mfloat-abi=soft -mno-unaligned-access -fstack-protector -Wno-unused-but-set-variable -o /Build/ArmPlatform/RELEASE_GCC48/ARM/ArmPkg/Library/BaseMemoryLibStm/BaseMemoryLibStm/OUTPUT/./ScanMem64Wrapper.obj -I/ArmPkg/Library/BaseMemoryLibStm/Arm -I/ArmPkg/Library/BaseMemoryLibStm -I/Build/ArmPlatform/RELEASE_GCC48/ARM/ArmPkg/Library/BaseMemoryLibStm/BaseMemoryLibStm/DEBUG -I/MdePkg -I/MdePkg/Include -I/MdePkg/Include/Arm /ArmPkg/Library/BaseMemoryLibStm/ScanMem64Wrapp er.c /arm/tools/arm/gcc-arm-none-eabi/4_8-2014q2/bin/../lib/gcc/arm-none-eabi/4.8.4/../../../../arm-none-eabi/bin/ld: cannot find -lssp_nonshared /arm/tools/arm/gcc-arm-none-eabi/4_8-2014q2/bin/../lib/gcc/arm-none-eabi/4.8.4/../../../../arm-none-eabi/bin/ld: cannot find -lssp collect2: error: ld returned 1 exit status I have not investigated much this issue, but I would not be surprise if it is due to the fact you removed '-c' from 'GCC_ALL_CC_FLAGS'. ________________________________________ From: Scott Duplichan [[email protected]] Sent: 07 November 2014 06:05 To: [email protected] Subject: [edk2] Status of gcc link time optimization for EDK2 use http://notabs.org/uefi/gcc-lto.htm Thanks, Scott ------------------------------------------------------------------------------ _______________________________________________ edk2-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-devel -- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. ARM Limited, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No: 2557590 ARM Holdings plc, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No: 2548782 ------------------------------------------------------------------------------ _______________________________________________ edk2-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-devel ------------------------------------------------------------------------------ Comprehensive Server Monitoring with Site24x7. Monitor 10 servers for $9/Month. Get alerted through email, SMS, voice calls or mobile push notifications. Take corrective actions from your mobile device. http://pubads.g.doubleclick.net/gampad/clk?id=154624111&iu=/4140/ostg.clktrk _______________________________________________ edk2-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-devel
