[PATCH] D53384: [PowerPC] Make no-PIC default to match GCC - CLANG

[Sean Fertile via Phabricator via llvm-commits]

sfertile added inline comments.


================
Comment at: test/Driver/print-multi-directory.c:7
 
-// CHECK-X86-MULTILIBS:      32
+// CHECK-X86-MULTILIBS:      .
 // CHECK-X86-MULTILIBS-NOT:  {{^.+$}}
----------------
stefanp wrote:
> jsji wrote:
> > Do we know why this changes due to fPIC default change?  
> > If this is a non-relevant change, maybe deliver in another NFC commit? Thanks.
> No I don't know exactly why this changes.
> I was actually debating in this situation whether or not I should just add -fPIC to the options to keep the behaviour the same as before. 
> 
> Unfortunately I can't put this in a later NFC change because I would leave a failing test. I either have to adjust the test or I would have to add -fPIC. 
This test changing  behavior is suspect, the run line uses a target that is not powerpc and if the test somehow depended on the hosts default pic value wouldn't it be failing for mips64 as well? FWIW I applied this change to top-of-trunk and ran the test an X86 machine and the test behavior didn't change. I think it worth investigating to find out why this test changes behavior due to this change.


https://reviews.llvm.org/D53384