Bruce Cran [mailto:bruce.c...@gmail.com]  wrote:

]On 11/3/2014 9:21 AM, Laszlo Ersek wrote:
]
]> So Scott's patch seems to be aligned with the
]> tradition. (Currently no gcc optimizations are enabled at all when
]> building for X64, neither for speed nor for size.)
]
]Doesn't the following cause X64 builds to use -Os (and IA32 to use -O2, 
]if it wasn't overridden by -Os by later, version-specific flags)?
]
]DEFINE GCC_ALL_CC_FLAGS            = -g -Os -fshort-wchar 
]-fno-strict-aliasing -Wall -Werror -Wno-array-bounds -c -include AutoGen.h
]
]DEFINE GCC_IA32_CC_FLAGS           = DEF(GCC_ALL_CC_FLAGS) -m32 
]-malign-double -freorder-blocks -freorder-blocks-and-partition -O2 
]-mno-stack-arg-probe
]
]DEFINE GCC_X64_CC_FLAGS            = DEF(GCC_ALL_CC_FLAGS) -mno-red-zone 
]-Wno-address -mno-stack-arg-probeq
]
]-- 
]Bruce

I believe you are right for the case of UNIXGCC and CYGGCC X64. ELFGCC
X64 also uses -Os. But I happened to be testing with GCC49 X64, and that
one doesn't use any -O flag. Reducing/consolidating the various gcc tool
chain definitions would be helpful.

Thanks,
Scott


------------------------------------------------------------------------------
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to