Understood, The test worked for me because I have both 64 bit and 32 bit libs 
installed on my default sysroot. Otherwise might fail indeed.

Will amend the test with an explicit sysroot, 


> -----Original Message-----
> From: Tim Shen via Phabricator [mailto:revi...@reviews.llvm.org]
> Sent: Wednesday, September 05, 2018 12:12 AM
> To: Christian BRUEL <christian.br...@st.com>; jroel...@jroelofs.com
> Cc: tims...@google.com; srhi...@google.com; cfe-commits@lists.llvm.org
> Subject: [PATCH] D51354: Fix the -print-multi-directory flag to print the
> selected multilib.
> 
> 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
> 
> 

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to