<font size=2 face="sans-serif">Hi Aaron, </font><br><br><font size=2 face="sans-serif">Sorry about giving so little info.</font><br><br><font size=2 face="sans-serif">The commit was reverted because it broke
two test cases on an internal buildbot. The reason this was reverted so
late was because this failure was buried underneath another set of failures
on that same buildbot which initially hid the problem. I'm looking at the
failures now and hopefully I'll have the change back in soon.</font><br><br><font size=2 face="sans-serif">Stefan</font><br><br><br><br><font size=1 color=#5f5f5f face="sans-serif">From:      
 </font><font size=1 face="sans-serif">Aaron Ballman <aaron@aaronballman.com></font><br><font size=1 color=#5f5f5f face="sans-serif">To:      
 </font><font size=1 face="sans-serif">stefanp@ca.ibm.com</font><br><font size=1 color=#5f5f5f face="sans-serif">Cc:      
 </font><font size=1 face="sans-serif">cfe-commits <cfe-commits@lists.llvm.org></font><br><font size=1 color=#5f5f5f face="sans-serif">Date:      
 </font><font size=1 face="sans-serif">2018/12/11 10:56 AM</font><br><font size=1 color=#5f5f5f face="sans-serif">Subject:    
   </font><font size=1 face="sans-serif">Re: r348858
- Revert "[PowerPC] Make no-PIC default to match GCC - CLANG"</font><br><hr noshade><br><br><br><tt><font size=2>On Tue, Dec 11, 2018 at 10:50 AM Stefan Pintilie via
cfe-commits<br><cfe-commits@lists.llvm.org> wrote:<br>><br>> Author: stefanp<br>> Date: Tue Dec 11 07:47:57 2018<br>> New Revision: 348858<br>><br>> URL: </font></tt><a href="http://llvm.org/viewvc/llvm-project?rev=348858&view=rev"><tt><font size=2>http://llvm.org/viewvc/llvm-project?rev=348858&view=rev</font></tt></a><tt><font size=2><br>> Log:<br>> Revert "[PowerPC] Make no-PIC default to match GCC - CLANG"<br>><br>> This reverts commit rL348299.<br><br>When reverting a commit, you should explain why the commit was<br>reverted as part of the commit message (this makes code archaeology<br>much easier). Why was this reverted 500+ revisions after it landed?<br><br>~Aaron<br><br>><br>> Modified:<br>>     cfe/trunk/lib/Driver/ToolChains/Gnu.cpp<br>>     cfe/trunk/test/Driver/clang-offload-bundler.c<br>>     cfe/trunk/test/Driver/ppc-abi.c<br>><br>> Modified: cfe/trunk/lib/Driver/ToolChains/Gnu.cpp<br>> URL: </font></tt><a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains/Gnu.cpp?rev=348858&r1=348857&r2=348858&view=diff"><tt><font size=2>http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains/Gnu.cpp?rev=348858&r1=348857&r2=348858&view=diff</font></tt></a><tt><font size=2><br>> ==============================================================================<br>> --- cfe/trunk/lib/Driver/ToolChains/Gnu.cpp (original)<br>> +++ cfe/trunk/lib/Driver/ToolChains/Gnu.cpp Tue Dec 11 07:47:57 2018<br>> @@ -2435,7 +2435,7 @@ bool Generic_GCC::isPICDefault() const {<br>>    case llvm::Triple::x86_64:<br>>      return getTriple().isOSWindows();<br>>    case llvm::Triple::ppc64:<br>> -    // Big endian PPC is PIC by default<br>> +  case llvm::Triple::ppc64le:<br>>      return !getTriple().isOSBinFormatMachO() &&
!getTriple().isMacOSX();<br>>    case llvm::Triple::mips64:<br>>    case llvm::Triple::mips64el:<br>><br>> Modified: cfe/trunk/test/Driver/clang-offload-bundler.c<br>> URL: </font></tt><a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/clang-offload-bundler.c?rev=348858&r1=348857&r2=348858&view=diff"><tt><font size=2>http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/clang-offload-bundler.c?rev=348858&r1=348857&r2=348858&view=diff</font></tt></a><tt><font size=2><br>> ==============================================================================<br>> --- cfe/trunk/test/Driver/clang-offload-bundler.c (original)<br>> +++ cfe/trunk/test/Driver/clang-offload-bundler.c Tue Dec 11 07:47:57
2018<br>> @@ -115,7 +115,7 @@<br>>  // CK-TEXTI: // __CLANG_OFFLOAD_BUNDLE____END__ openmp-x86_64-pc-linux-gnu<br>><br>>  // CK-TEXTLL: ; __CLANG_OFFLOAD_BUNDLE____START__ host-powerpc64le-ibm-linux-gnu<br>> -// CK-TEXTLL: @A = dso_local global i32 0<br>> +// CK-TEXTLL: @A = global i32 0<br>>  // CK-TEXTLL: define {{.*}}@test_func()<br>>  // CK-TEXTLL: ; __CLANG_OFFLOAD_BUNDLE____END__ host-powerpc64le-ibm-linux-gnu<br>>  // CK-TEXTLL: ; __CLANG_OFFLOAD_BUNDLE____START__ openmp-powerpc64le-ibm-linux-gnu<br>><br>> Modified: cfe/trunk/test/Driver/ppc-abi.c<br>> URL: </font></tt><a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/ppc-abi.c?rev=348858&r1=348857&r2=348858&view=diff"><tt><font size=2>http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/ppc-abi.c?rev=348858&r1=348857&r2=348858&view=diff</font></tt></a><tt><font size=2><br>> ==============================================================================<br>> --- cfe/trunk/test/Driver/ppc-abi.c (original)<br>> +++ cfe/trunk/test/Driver/ppc-abi.c Tue Dec 11 07:47:57 2018<br>> @@ -13,12 +13,12 @@<br>>  // RUN: %clang -target powerpc64-unknown-linux-gnu %s -### -o
%t.o 2>&1 \<br>>  // RUN:   -mcpu=a2q -mno-qpx | FileCheck -check-prefix=CHECK-ELFv1
%s<br>>  // RUN: %clang -target powerpc64-unknown-linux-gnu %s -### -o
%t.o 2>&1 \<br>> -// RUN:   -mabi=elfv2 | FileCheck -check-prefix=CHECK-ELFv2-BE
%s<br>> +// RUN:   -mabi=elfv2 | FileCheck -check-prefix=CHECK-ELFv2
%s<br>><br>>  // RUN: %clang -target powerpc64le-unknown-linux-gnu %s -###
-o %t.o 2>&1 \<br>>  // RUN:   | FileCheck -check-prefix=CHECK-ELFv2 %s<br>>  // RUN: %clang -target powerpc64le-unknown-linux-gnu %s -###
-o %t.o 2>&1 \<br>> -// RUN:   -mabi=elfv1 | FileCheck -check-prefix=CHECK-ELFv1-LE
%s<br>> +// RUN:   -mabi=elfv1 | FileCheck -check-prefix=CHECK-ELFv1
%s<br>>  // RUN: %clang -target powerpc64le-unknown-linux-gnu %s -###
-o %t.o 2>&1 \<br>>  // RUN:   -mabi=elfv2 | FileCheck -check-prefix=CHECK-ELFv2
%s<br>>  // RUN: %clang -target powerpc64le-unknown-linux-gnu %s -###
-o %t.o 2>&1 \<br>> @@ -26,44 +26,8 @@<br>><br>>  // CHECK-ELFv1: "-mrelocation-model" "pic"
"-pic-level" "2"<br>>  // CHECK-ELFv1: "-target-abi" "elfv1"<br>> -// CHECK-ELFv1-LE: "-mrelocation-model" "static"<br>> -// CHECK-ELFv1-LE: "-target-abi" "elfv1"<br>>  // CHECK-ELFv1-QPX: "-mrelocation-model" "pic"
"-pic-level" "2"<br>>  // CHECK-ELFv1-QPX: "-target-abi" "elfv1-qpx"<br>> -// CHECK-ELFv2: "-mrelocation-model" "static"<br>> +// CHECK-ELFv2: "-mrelocation-model" "pic" "-pic-level"
"2"<br>>  // CHECK-ELFv2: "-target-abi" "elfv2"<br>> -// CHECK-ELFv2-BE: "-mrelocation-model" "pic"
"-pic-level" "2"<br>> -// CHECK-ELFv2-BE: "-target-abi" "elfv2"<br>> -<br>> -// RUN: %clang -fPIC -target powerpc64-unknown-linux-gnu %s -###
-o %t.o 2>&1 \<br>> -// RUN:   | FileCheck -check-prefix=CHECK-ELFv1-PIC %s<br>> -// RUN: %clang -fPIC -target powerpc64-unknown-linux-gnu %s -###
-o %t.o 2>&1 \<br>> -// RUN:   -mabi=elfv1 | FileCheck -check-prefix=CHECK-ELFv1-PIC
%s<br>> -// RUN: %clang -fPIC -target powerpc64-unknown-linux-gnu %s -###
-o %t.o 2>&1 \<br>> -// RUN:   -mabi=elfv1-qpx | FileCheck -check-prefix=CHECK-ELFv1-QPX-PIC
%s<br>> -// RUN: %clang -fPIC -target powerpc64-unknown-linux-gnu %s -###
-o %t.o 2>&1 \<br>> -// RUN:   -mcpu=a2q | FileCheck -check-prefix=CHECK-ELFv1-QPX-PIC
%s<br>> -// RUN: %clang -fPIC -target powerpc64-unknown-linux-gnu %s -###
-o %t.o 2>&1 \<br>> -// RUN:   -mcpu=a2 -mqpx | FileCheck -check-prefix=CHECK-ELFv1-QPX-PIC
%s<br>> -// RUN: %clang -fPIC -target powerpc64-unknown-linux-gnu %s -###
-o %t.o 2>&1 \<br>> -// RUN:   -mcpu=a2q -mno-qpx | FileCheck -check-prefix=CHECK-ELFv1-PIC
%s<br>> -// RUN: %clang -fPIC -target powerpc64-unknown-linux-gnu %s -###
-o %t.o 2>&1 \<br>> -// RUN:   -mabi=elfv2 | FileCheck -check-prefix=CHECK-ELFv2-PIC
%s<br>> -<br>> -// RUN: %clang -fPIC -target powerpc64le-unknown-linux-gnu %s -###
-o %t.o 2>&1 \<br>> -// RUN:   | FileCheck -check-prefix=CHECK-ELFv2-PIC %s<br>> -// RUN: %clang -fPIC -target powerpc64le-unknown-linux-gnu %s -###
-o %t.o 2>&1 \<br>> -// RUN:   -mabi=elfv1 | FileCheck -check-prefix=CHECK-ELFv1-PIC
%s<br>> -// RUN: %clang -fPIC -target powerpc64le-unknown-linux-gnu %s -###
-o %t.o 2>&1 \<br>> -// RUN:   -mabi=elfv2 | FileCheck -check-prefix=CHECK-ELFv2-PIC
%s<br>> -// RUN: %clang -fPIC -target powerpc64le-unknown-linux-gnu %s -###
-o %t.o 2>&1 \<br>> -// RUN:   -mabi=altivec | FileCheck -check-prefix=CHECK-ELFv2-PIC
%s<br>> -<br>> -// CHECK-ELFv1-PIC: "-mrelocation-model" "pic"
"-pic-level" "2"<br>> -// CHECK-ELFv1-PIC: "-target-abi" "elfv1"<br>> -// CHECK-ELFv1-QPX-PIC: "-mrelocation-model" "pic"
"-pic-level" "2"<br>> -// CHECK-ELFv1-QPX-PIC: "-target-abi" "elfv1-qpx"<br>> -// CHECK-ELFv2-PIC: "-mrelocation-model" "pic"
"-pic-level" "2"<br>> -// CHECK-ELFv2-PIC: "-target-abi" "elfv2"<br>> -<br>><br>><br>><br>> _______________________________________________<br>> cfe-commits mailing list<br>> cfe-commits@lists.llvm.org<br>> </font></tt><a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits"><tt><font size=2>http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</font></tt></a><tt><font size=2><br><br></font></tt><br><BR>