[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