diff options
author | Joseph Huber <jhuber6@vols.utk.edu> | 2021-12-09 16:10:46 -0500 |
---|---|---|
committer | Joseph Huber <jhuber6@vols.utk.edu> | 2021-12-09 16:30:29 -0500 |
commit | bc9c4d7216a5bcdeec8543834fe1451db2c46e5c (patch) | |
tree | 31bb3b4f17c58781eebc971bef31722428a9994f /clang/lib/Frontend/CompilerInvocation.cpp | |
parent | ad3bb7c7da7b9e66cb98629d0f579546c8ead48f (diff) | |
download | llvm-bc9c4d7216a5bcdeec8543834fe1451db2c46e5c.zip llvm-bc9c4d7216a5bcdeec8543834fe1451db2c46e5c.tar.gz llvm-bc9c4d7216a5bcdeec8543834fe1451db2c46e5c.tar.bz2 |
[OpenMP][FIX] Pass the num_threads value directly to parallel_51
The problem with the old scheme is that we would need to keep track of
the "next region" and reset the num_threads value after it. The new RT
doesn't do it and an assertion is triggered. The old RT doesn't do it
either, I haven't tested it but I assume a num_threads clause might
impact multiple parallel regions "accidentally". Further, in SPMD mode
num_threads was simply ignored, for some reason beyond me.
In any case, parallel_51 is designed to take the clause value directly,
so let's do that instead.
Reviewed By: tianshilei1992
Differential Revision: https://reviews.llvm.org/D113623
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions