<div dir="ltr">i686-pc-linux-gnu<div><br></div><div>I can attach more logs if you need them?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 22, 2015 at 10:36 AM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">But not phase 1? Interesting. What target does clang --version on the<br>
phase 1 compiler print?<br>
<span class=""><br>
On Tue, Jul 21, 2015 at 3:36 PM, Nikola Smiljanic <<a href="mailto:popizdeh@gmail.com">popizdeh@gmail.com</a>> wrote:<br>
> Building phase 2 fails on i686 Fedora 22<br>
><br>
> CMake Error at projects/compiler-rt/cmake/config-ix.cmake:125 (message):<br>
>   Cannot compile for i686:<br>
><br>
> CMakeError.log attached<br>
><br>
</span><span class="">> On Wed, Jul 22, 2015 at 8:20 AM, Alexey Samsonov <<a href="mailto:vonosmas@gmail.com">vonosmas@gmail.com</a>> wrote:<br>
>><br>
>> The problem is we "guess" that the host architecture is i686, and fail<br>
>> when we find out that we can't target it (__i686__ is not defined).<br>
>> This guess originates from get_host_triple() call in<br>
>> cmake/modules/GetHostTriple.cmake.<br>
>> Probably, it makes sense to fix that to return i568 on openSUSE, and then<br>
>> test for i586 in the same way we already test for i386/i686.<br>
>><br>
>><br>
</span><span class="">>> On Fri, Jul 17, 2015 at 11:23 AM, Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br>
>>><br>
>>> Seems on OpenSUSE x86, it's called i586, not i686 :-(<br>
>>><br>
>>> +Alexey: do you think we can handle this in the compiler-rt cmake<br>
>>> files somehow? Maybe try targeting both i686 and i586 unless that<br>
>>> would break something else?<br>
>>><br>
</span>>>> On Fri, Jul 17, 2015 at 1:31 AM, Nikola Smiljanic <<a href="mailto:popizdeh@gmail.com">popizdeh@gmail.com</a>><br>
<span class="">>>> wrote:<br>
>>> > CMake Error at projects/compiler-rt/cmake/config-ix.cmake:125<br>
>>> > (message):<br>
>>> > Cannot compile for i686<br>
>>> ><br>
>>> > CMakeError.log attached, seems like #include checks are failing? This<br>
>>> > is x86<br>
>>> > openSUSE.<br>
>>> ><br>
</span>>>> > On Fri, Jul 17, 2015 at 9:14 AM, Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>><br>
<span class="">>>> > wrote:<br>
>>> >><br>
>>> >> Hi Jack,<br>
>>> >><br>
>>> >> On Thu, Jul 16, 2015 at 4:03 PM, Jack Howarth<br>
</span><div><div class="h5">>>> >> <<a href="mailto:howarth.mailing.lists@gmail.com">howarth.mailing.lists@gmail.com</a>> wrote:<br>
>>> >> > Hans,<br>
>>> >> >       Do we intend to leave -fopenmp defaulted to the no-op libgomp<br>
>>> >> > support for 3.7.0 or do the sensible thing by applying...<br>
>>> >> ><br>
>>> >> > Index: CMakeLists.txt<br>
>>> >> > ===================================================================<br>
>>> >> > --- CMakeLists.txt (revision 242425)<br>
>>> >> > +++ CMakeLists.txt (working copy)<br>
>>> >> > @@ -182,7 +182,7 @@ set(GCC_INSTALL_PREFIX "" CACHE PATH "Di<br>
>>> >> >  set(DEFAULT_SYSROOT "" CACHE PATH<br>
>>> >> >    "Default <path> to all compiler invocations for<br>
>>> >> > --sysroot=<path>." )<br>
>>> >> ><br>
>>> >> > -set(CLANG_DEFAULT_OPENMP_RUNTIME "libgomp" CACHE STRING<br>
>>> >> > +set(CLANG_DEFAULT_OPENMP_RUNTIME "libomp" CACHE STRING<br>
>>> >> >    "Default OpenMP runtime used by -fopenmp.")<br>
>>> >> ><br>
>>> >> >  set(CLANG_VENDOR "" CACHE STRING<br>
>>> >> ><br>
>>> >> > so that the new llvm openmp library (which passes a three stage<br>
>>> >> > bootstrap as part of an<br>
>>> >> > in-tree build of llvm/cfe/compiler-rt/polly/libc++/openmp on<br>
>>> >> > x86_64-apple-darwin).<br>
>>> >><br>
>>> >> I'm not fully aware of the implications of this, but if we do want to<br>
>>> >> change it, it needs to be done on trunk first.<br>
>>> >><br>
>>> >> If you get it through review and committed to trunk, I'm open to<br>
>>> >> merging<br>
>>> >> it.<br>
>>> >><br>
>>> >> I assume utils/release/test-release.sh would also need an update so<br>
>>> >> the library gets built?<br>
>>> >><br>
>>> >> Thanks,<br>
>>> >> Hans<br>
>>> ><br>
>>> ><br>
>><br>
>><br>
>><br>
>><br>
>> --<br>
>> Alexey Samsonov<br>
</div></div>>> <a href="mailto:vonosmas@gmail.com">vonosmas@gmail.com</a><br>
><br>
><br>
</blockquote></div><br></div>