[doc] Use correct OMP_PROC_BIND arguments in the tutorial examples #3435
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
The example to use Stream seems to use
OMP_PROC_BINDbut sets possible values asclose,cores,spread. This confused me and therefore suggest the following unless I am missing something.coresdoes seem to be a valid option forOMP_PROC_BINDand only forOMP_PLACES.trueis a valid option and doesnt enforce a method for affinity. Could also befalseto allow threads to migrate.For example see: https://www.openmp.org/spec-html/5.0/openmpse52.html#x291-20580006.4
Is there another reason why it's set to
coresas part of the learning experience? I couldn't see any reference to why it would be.