Opened 9 years ago
Closed 9 years ago
#15086 closed defect (wontfix)
XPCOM Config.kmk should use -O2 and -std=c++11
Reported by: | brogan | Owned by: | |
---|---|---|---|
Component: | other | Version: | VirtualBox 5.0.14 |
Keywords: | xpcom, xpcom18a4, Config.kmk | Cc: | |
Guest type: | all | Host type: | Linux |
Description
In file https://www.virtualbox.org/browser/vbox/trunk/src/libs/xpcom18a4/Config.kmk
a) -O needs to be changed to -O2 b) -ansi needs to be changed to -std=c++11
Thanks.
Note:
See TracTickets
for help on using tickets.
This is actually old code not suitable for compiling with -O2. In particular the code is most likely not safe for gcc strict-aliasing rules. Replacing -O with -O2 would not buy anything except the risk of a few not obvious bugs.
I also don't see the need for -std=c++11. This switch is not supported on older gcc versions. We still build a couple of packages on EL5 with gcc 4.1.1.