[PATCH] D33804: bugpoint: Prototype disabling symbolication of bugpoint-executed programs

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 8 21:15:02 PDT 2017


On Thu, Jun 8, 2017 at 8:57 PM, David Blaikie <dblaikie at gmail.com> wrote:

>
>
> On Thu, Jun 8, 2017 at 8:11 PM Daniel Berlin via Phabricator <
> reviews at reviews.llvm.org> wrote:
>
>> dberlin accepted this revision.
>> dberlin added a comment.
>> This revision is now accepted and ready to land.
>>
>> I think this is the right thing to do given what we've seen.
>
>
> Dandy - thanks. I'll see if I can figure out some test coverage for this
> (find a way to crash things, ensure the test works in all configurations,
> etc) check it in and then add in the support for the other commands
> (llc/lli I think) that bugpoint runs directly.
>
>
>> Nobody has come up with a use case where they current crawl backtraces in
>> their scripts, etc.
>>
>
> In their scripts it'll be up to them to pass or not pass this flag (does
> mean if they want speed they'll have to know about this flag to pass it to
> avoid the symbolizing overhead).
>

It looked like you spawn bugpoint passes with it?
(IE when do they have to pass it)

I'm referring to this:
bool BugDriver::runPasses(Module *Program,
                          const std::vector<std::string> &Passes,
                          std::string &OutputFilename, bool DeleteOutput,
                          bool Quiet, unsigned NumExtraArgs,
                          const char *const *ExtraArgs) const {
...

  Args.push_back("-disable-symbolication");
...
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170608/e7d318f8/attachment.html>


More information about the llvm-commits mailing list