This is an archive of the discontinued LLVM Phabricator instance.

[PowerPC] PPC LLD buildbot fix previous patch
ClosedPublic

Authored by stefanp on Sep 16 2019, 1:04 PM.

Details

Summary

It looks like the previous patch is causing errors on the bot due to option parsing.
Previous patch is here.
https://reviews.llvm.org/D65820
This is the error that is produced: llvm-lit: error: argument -v/--verbose: ignored explicit argument ' --threads=10'

I've changed it so that there are no more issues by using -j 10 instead. It seems that the quotes around the parameters means it tries to take everything as one argument. Bu using -j we can actually pass everything as one argument by attaching the j to the sv.

As a side note to this, is there anywhere we can test zorg changes before adding patches like this?

Diff Detail

Event Timeline

stefanp created this revision.Sep 16 2019, 1:04 PM
This revision is now accepted and ready to land.Sep 16 2019, 1:52 PM

@gkistanova
Thank you for reviewing this Galina.
I don't have commit access to zorg. Can you commit this for me?

stefanp closed this revision.Sep 19 2019, 11:32 AM

Thank you Galina!
Committed at r372342.