Issue 72289 - ENABLE_CRASHDUMP disables PCH usage
Summary: ENABLE_CRASHDUMP disables PCH usage
Status: CLOSED FIXED
Alias: None
Product: Build Tools
Classification: Code
Component: solenv (show other issues)
Version: current
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: hjs
QA Contact: issues@tools
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-12-05 12:07 UTC by hjs
Modified: 2013-08-07 15:34 UTC (History)
3 users (show)

See Also:
Issue Type: TASK
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments
proposed fix (1.14 KB, patch)
2007-01-08 12:23 UTC, hjs
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this issue.
Description hjs 2006-12-05 12:07:58 UTC
the additional compiler switches in case of ENABLE_CRASHDUMP cause the sanity
check for the used compiler switches to fail and thus disable the PCH usage.
Comment 1 hjs 2006-12-05 13:46:03 UTC
it's even worse:
filtering the according compiler switches (-Zi
-Fd$(MISC)$/_ooo_st_$(TARGET).PDB) from the compared strings reenables PCH
usage. unfortunately the compiler now refuses to use the generated *.pch file as
the .PDB name stored inside the .pch file differs from that used to compile the
c++ file...
Comment 2 hjs 2006-12-05 14:35:02 UTC
it may be an option to use "-Z7 -Yd" insted of "-Zi
-Fd$(MISC)$/_ooo_st_$(TARGET).PDB" and get rid of the .pdb files in the compile
step completely.
Comment 3 hjs 2006-12-18 10:03:12 UTC
as requested
Comment 4 hennes.rohling 2006-12-21 10:02:12 UTC
Crash report resolving along with /Z7 looks good in module VCL.

According to MSDN documentation /Yc /Yd might cause link error if procompiled
headers don't contain function definitions. Missing symbols have to be specified
by /YI. To ensure that this will not cause trouble a complete rebuild of all
modules (build OOo from scratch) is necessary.

When using /Z7 instead of /Zi all .obj files must be available for JIT debugging
with /Zi the .PCB with type infos in wntmsci10.pro/misc were suitable. But this
will not affect the build as the complete source output tree is always available
in OOo's build environment.
Comment 5 hjs 2007-01-08 12:23:38 UTC
Created attachment 41995 [details]
proposed fix
Comment 6 hjs 2007-01-08 12:24:59 UTC
this issue depends on cws pchfix04 as it will bring wider usage of pch
Comment 7 hjs 2007-01-26 09:53:16 UTC
patch applied in pchfix04 now. ENABLE_CRASHDUMP and PCH now coexist.

complete build dtill pending
Comment 8 hjs 2007-02-02 10:12:22 UTC
unfortunately these switches cause some of the large .lib files used to detect
library exports to hit some kind of size limit :(

tweaking process to support more than on lib and splitting candidates into two
pieces...
Comment 9 hjs 2007-02-05 12:58:02 UTC
.
Comment 10 gregor.hartmann 2007-03-02 14:21:01 UTC
useing "-Z7 -Yd"
Comment 11 rt 2007-05-30 13:10:32 UTC
Seen on MWS, closing