scons icon indicating copy to clipboard operation
scons copied to clipboard

Support for precompiled headers in intelc

Open bdbaddog opened this issue 7 years ago • 1 comments

This issue was originally created at: 2007-10-25 06:37:51. This issue was reported by: savardc.

savardc said at 2007-10-25 06:37:51

[Editorial: request to add PCH for Intel compilers, as of 2024 the syntax is documented at the following llink: ] https://www.intel.com/content/www/us/en/docs/dpcpp-cpp-compiler/developer-guide-reference/2024-2/output-debug-and-precompiled-header-options.html

I added the support for pch in intelc. It's basically a port of what's in msvc.py but I needed to add 2 additional builders (SharedPCH, StaticPCH) in order to support share and static objects. I kept the old builder (PCH) for backward compability.

savardc said at 2007-10-25 06:38:24 Created an attachment (id=234)

Patch for intelc.py

savardc said at 2007-10-25 06:38:48 Created an attachment (id=235)

Patch for msvc.py

garyo said at 2007-10-26 07:39:24

Hi Christian -- thanks for this. I'd like to integrate this (it may be a few days til I can get to it). Could you add a test case for it, like the msvc ones? Also if you could update the doc that would be even better (but if not, I will.)

This should work on Linux and Windows, right?

issues@scons said at 2007-10-26 08:11:48 Created an attachment (id=240)

Fixed a typo in the patch

savardc said at 2007-10-26 09:53:39 Created an attachment (id=241)

Correction to the msvc doc for the new builders and intel support

gregnoel said at 2008-05-30 13:34:01

Bug party triage.

gregnoel said at 2008-12-26 13:29:16

Adjust triage of issues.

savardc attached intelc.patch at 2007-10-25 06:38:23.

Patch for intelc.py

savardc attached msvc.patch at 2007-10-25 06:38:48.

Patch for msvc.py

An anonymous user attached msvc.patch at 2007-10-26 08:11:48.

Fixed a typo in the patch

savardc attached msvc.xml.patch at 2007-10-26 09:53:39.

Correction to the msvc doc for the new builders and intel support

bdbaddog avatar Jan 02 '18 11:01 bdbaddog

It seems like this is asking for the same thing as #1385 (if we merge them, I'd keep this one since it proposed a patch, even if that's probably outdated by now)

mwichmann avatar Jul 29 '21 19:07 mwichmann