Skip to content

Make PTG compiler optional#783

Closed
devreal wants to merge 1 commit into
ICLDisco:masterfrom
devreal:runtime-only
Closed

Make PTG compiler optional#783
devreal wants to merge 1 commit into
ICLDisco:masterfrom
devreal:runtime-only

Conversation

@devreal

@devreal devreal commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Not all consumers of PaRSEC require the PTG compiler. Setting PARSEC_BUILD_PTGPP=OFF will build only the runtime component needed for DTD and direct users of the PaRSEC runtime.

Also fixes a duplicated installation of apply.h.

Not all consumers of PaRSEC require the PTG compiler.
Setting `PARSEC_BUILD_PTGPP=OFF` will build only the runtime component
needed for DTD and direct users of the PaRSEC runtime.

Also fixes a duplicated installation of apply.h.

Signed-off-by: Joseph Schuchart <joseph.schuchart@stonybrook.edu>
@devreal devreal requested a review from a team as a code owner June 4, 2026 12:18

@bosilca bosilca left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you also need to remove it from the generated PaRSECConfig.cmake to let any potential users know the ptgpp is missing in this install.

My big problem with this is that you now get a crippled data helper library, everything based on JDF will not exists, such as basic collectives, reduce/broadcast, or more elaborate features, the data redistribution.

So I guess the real question is what is the harm of builbin the ptgcc most of the time, even if you don't plan to use it ?

@devreal

devreal commented Jun 4, 2026

Copy link
Copy Markdown
Contributor Author

Mhh, I was having issues with executing the PTG compiler during build when enabling address sanitizer. Not because of an issue in PaRSEC but because ASAN is stuck in init. Then I thought that I don't really need the compiler. But having an incomplete PaRSEC library also isn't great. Closing for now.

@devreal devreal closed this Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants