[PATCH] D51354: Fix the -print-multi-directory flag to print the selected multilib.

Christian BRUEL via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 5 04:36:25 PDT 2018


Hello all,

Is there a way to see how the patch looked like after my commit and 
before the revert from the git mirror ?

I cannot reproduce the test failure from the bots in my local build, 
(which is based on the the git mirror). but I noticed that a similar 
behavior however occurs when the test patch is applied twice. (I have 
exactly the same error if I duplicate the lines in print-multi-directory.c)

I'm curious to see if there was a difference between the review D51354 
patch and the actual svn commit with arcanist that I might not have used 
correctly.

That was actually my first commit. What I did was from a clean 
subversion update

arc patch D51354
arc commit --revision D51354

I'm not sure what could be wrong, any idea ?

thanks

Christian

On 09/05/2018 12:11 AM, Tim Shen via Phabricator wrote:
> timshen added a comment.
>
>> The test fails on my system like so:
> I also observed the same failure.
>
> Bots also fail: http://lab.llvm.org:8011/builders/clang-x86_64-linux-selfhost-modules/builds/19509/steps/check-all/logs/FAIL%3A%20Clang%3A%3Aprint-multi-directory.c
>
> I'm going to revert this patch.
>
>
> Repository:
>    rC Clang
>
> https://reviews.llvm.org/D51354
>
>
>


More information about the cfe-commits mailing list