diff options
author | Olexa Bilaniuk <obilaniu@gmail.com> | 2021-03-12 05:51:44 -0500 |
---|---|---|
committer | Nirbheek Chauhan <nirbheek@centricular.com> | 2021-04-04 00:09:08 +0530 |
commit | 1ae42e8842187d17208f8010718379d7d501cb40 (patch) | |
tree | 05556d3657f12913b229df718fa6decfe87b80fe | |
parent | b03778f6f68837d1e37e05e83bee969488491e2f (diff) | |
download | meson-1ae42e8842187d17208f8010718379d7d501cb40.zip meson-1ae42e8842187d17208f8010718379d7d501cb40.tar.gz meson-1ae42e8842187d17208f8010718379d7d501cb40.tar.bz2 |
Canonicalize and merge consecutive -Xcompiler flags together.
Makes command-line more readable.
-rw-r--r-- | mesonbuild/compilers/cuda.py | 48 |
1 files changed, 47 insertions, 1 deletions
diff --git a/mesonbuild/compilers/cuda.py b/mesonbuild/compilers/cuda.py index bbe6e95..316143a 100644 --- a/mesonbuild/compilers/cuda.py +++ b/mesonbuild/compilers/cuda.py @@ -259,6 +259,52 @@ class CudaCompiler(Compiler): return '\,'.join(l) @classmethod + def _merge_flags(cls, flags: T.List[str]) -> T.List[str]: + r""" + The flags to NVCC gets exceedingly verbose and unreadable when too many of them + are shielded with -Xcompiler. Merge consecutive -Xcompiler-wrapped arguments + into one. + """ + if len(flags) <= 1: + return flags + flagit = iter(flags) + xflags = [] + + def is_xcompiler_flag_isolated(flag: str) -> bool: + return flag == '-Xcompiler' + def is_xcompiler_flag_glued(flag: str) -> bool: + return flag.startswith('-Xcompiler=') + def is_xcompiler_flag(flag: str) -> bool: + return is_xcompiler_flag_isolated(flag) or is_xcompiler_flag_glued(flag) + def get_xcompiler_val(flag: str, flagit: T.Iterator[str]) -> str: + if is_xcompiler_flag_glued(flag): + return flag[len('-Xcompiler='):] + else: + try: + return next(flagit) + except StopIteration: + return "" + + ingroup = False + for flag in flagit: + if not is_xcompiler_flag(flag): + ingroup = False + xflags.append(flag) + elif ingroup: + xflags[-1] += ',' + xflags[-1] += get_xcompiler_val(flag, flagit) + elif is_xcompiler_flag_isolated(flag): + ingroup = True + xflags.append(flag) + xflags.append(get_xcompiler_val(flag, flagit)) + elif is_xcompiler_flag_glued(flag): + ingroup = True + xflags.append(flag) + else: + raise ValueError("-Xcompiler flag merging failed, unknown argument form!") + return xflags + + @classmethod def _to_host_flags(cls, flags: T.List[str], phase: _Phase = _Phase.COMPILER) -> T.List[str]: """ Translate generic "GCC-speak" plus particular "NVCC-speak" flags to NVCC flags. @@ -431,7 +477,7 @@ class CudaCompiler(Compiler): xflags.append(flag) xflags.append(val) - return xflags + return cls._merge_flags(xflags) def needs_static_linker(self) -> bool: return False |