[Lldb-commits] [PATCH] D19998: Add a "-gmodules" category to the test suite.

Todd Fiala via lldb-commits lldb-commits at lists.llvm.org
Tue May 24 17:28:19 PDT 2016


tfiala added a comment.

In http://reviews.llvm.org/D19998#438614, @zturner wrote:

> In http://reviews.llvm.org/D19998#438586, @tfiala wrote:
>
> > F1982070: gmodules-test-support-v6.diff <http://reviews.llvm.org/F1982070>
> >
> > Updated gmodules.is_compiler_clang_with_gmodules() to guard on Windows with:
> >
> >   def _gmodules_supported_internal():
> >       compiler = os.path.basename(compiler_path)
> >       if "clang" not in compiler:
> >           return False
> >       elif os.name == "nt":
> >           # gmodules support is broken on Windows
> >           return False
> >       else:
> >           # Check the compiler help for the -gmodules option.
> >           clang_help = os.popen("%s --help" % compiler_path).read()
> >           return GMODULES_HELP_REGEX.search(clang_help, re.DOTALL) is not None
> >   
>
>
> So I asked some of the guys here, and they said modules debug info (in particular -gmodules) will not work anywhere but OSX.


I don't think that's right.  I ran all these tests on Linux, and our debug info guy (Adrian Prantl) thinks it is better supported on Linux than OS X.  ?  I'll let Adrian weigh in on that though.  I got all but the one dead code stripping test to run fine on Ubuntu 14.04 x86_64.


http://reviews.llvm.org/D19998





More information about the lldb-commits mailing list