[llvm] Adding Separate OpenMP Offloading Backend to `libcxx/include/__algorithm/pstl_backends` (PR #66968)
Louis Dionne via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 30 13:27:54 PDT 2023
================
@@ -330,6 +330,23 @@ def getStdFlag(cfg, std):
default=f"{shlex.quote(sys.executable)} {shlex.quote(str(Path(__file__).resolve().parent.parent.parent / 'run.py'))}",
help="Custom executor to use instead of the configured default.",
actions=lambda executor: [AddSubstitution("%{executor}", executor)],
+ ),
+ Parameter(
+ name="openmp_pstl_backend",
+ choices=[True, False],
+ type=bool,
+ default=False,
+ help="Enable the OpenMP compilation toolchain if the PSTL backend was set to OpenMP.",
+ actions=lambda enabled: [
+ AddCompileFlag("-fopenmp"),
----------------
ldionne wrote:
The only thing we should have to do here is add `-fopenmp`, assuming the compiler supports OpenMP. It should not be necessary to add custom link and include flags to use this. If it is necessary to do that, I would argue the compiler should be fixed to work out of the box. Do people using `-fopenmp` also specify these paths? That would be really surprising no?
https://github.com/llvm/llvm-project/pull/66968
More information about the llvm-commits
mailing list