[PATCH] D43601: [bugpoint] Add NoStripSymbols option.

Don Hinton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 21 17:04:32 PST 2018


hintonda added a comment.

In https://reviews.llvm.org/D43601#1015344, @vsk wrote:

> In https://reviews.llvm.org/D43601#1015343, @hintonda wrote:
>
> > In https://reviews.llvm.org/D43601#1015341, @vsk wrote:
> >
> > > A simpler way to do this might be to call 'strip' on the module as an final cleanup, just like we do for GlobalDCE.
> >
> >
> > So, always run it or use a flag?
>
>
> Gating the behavior behind an off-by-default flag sgtm.


You mean like the other "disable-strip*" options, right?

It would be unfortunate to have this one behave differently.


Repository:
  rL LLVM

https://reviews.llvm.org/D43601





More information about the llvm-commits mailing list