diff options
| author | Dylan Baker <dylan@pnwbakers.com> | 2019-03-18 15:52:04 -0700 |
|---|---|---|
| committer | Dylan Baker <dylan@pnwbakers.com> | 2019-03-19 12:48:22 -0700 |
| commit | 1f342a208122c86fa564688c518ecc63b67650df (patch) | |
| tree | 911d7f4bd82c633fdbd213a13afd3855da5ca31d /mesonbuild/backend/backends.py | |
| parent | 2a6b8e209fc6cbcd9499a0697c08bcb0169a2e54 (diff) | |
| download | meson-1f342a208122c86fa564688c518ecc63b67650df.zip meson-1f342a208122c86fa564688c518ecc63b67650df.tar.gz meson-1f342a208122c86fa564688c518ecc63b67650df.tar.bz2 | |
dependencies/openmp: Don't special case OpenMP
Currently we specialcase OpenMP like we do threads, with a special
`need_openmp` method. This seems like a great idea, but doesn't work
out in practice, as well as it complicates the opemp
implementation. If GCC is built without opemp support for example, we
still add -fopenmp to the the command line, which results in
compilation errors.
This patch discards that and treats it like a normal dependency,
removes the need_openmp() method, and sets the compile_args attributes
from the compiler.
Fixes #5115
Diffstat (limited to 'mesonbuild/backend/backends.py')
| -rw-r--r-- | mesonbuild/backend/backends.py | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/mesonbuild/backend/backends.py b/mesonbuild/backend/backends.py index 4d35d22..3ca2f99 100644 --- a/mesonbuild/backend/backends.py +++ b/mesonbuild/backend/backends.py @@ -641,8 +641,6 @@ class Backend: # pkg-config puts the thread flags itself via `Cflags:` if dep.need_threads(): commands += compiler.thread_flags(self.environment) - elif dep.need_openmp(): - commands += compiler.openmp_flags() # Fortran requires extra include directives. if compiler.language == 'fortran': for lt in target.link_targets: |