Use PCH
Bug #723667 reported by
Olaf van der Spek
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Drizzle |
Confirmed
|
Wishlist
|
Unassigned |
Bug Description
To decrease build times, A PCH should be used. A PCH (pch.h.gch) can be build with "g++ pch.h". Since I've no experience with the build system, could someone help me to add this bit?
A depedency on pch.h.gch should also be added to config.h.
Changed in drizzle: | |
importance: | Undecided → Wishlist |
status: | New → Confirmed |
milestone: | none → future |
To post a comment you must log in.
In a Hello World app, this suffices:
BUILT_SOURCES = config.h.gch
config.h.gch: config.h
${CXXCOMPILE} -o $@ $<
But in Drizzle this doesn't work.
It seems the GCC invocations don't match and the PCH is ignored.
PCH compile: inlines- hidden -Werror -pedantic -Wall -Wundef -Wshadow -fdiagnostics- show-option -floop- parallelize- all -fno-strict- aliasing -Wno-strict- aliasing -Wextra -Wformat -Wno-format- nonliteral -Wno-format- security -Woverloaded- virtual -Wnon-virtual-dtor -Wctor-dtor-privacy -Wno-long-long -Wmissing- declarations -Wframe- larger- than=32768 -Wno-redundant- decls -c config.h
g++ -DHAVE_CONFIG_H -I. -I. -g -pipe -std=gnu++98 -O2 -fvisibility=hidden -fvisibility-
Normal compile: message/ drizzled_ message_ libdrizzledmess age_la- statement_ transform. lo
CXX drizzled/
Monty, could you look into this?
BTW, why does it use .lo instead of .o?