[SYCL] Compilation Error when building the compiler for x86 target #646
Comments
|
Oh I should have said this is if you just type |
|
Actually both errors seem to appear. The first one regarding |
|
@crtrott, could check that your environment satisfies these pre-requisites - https://github.com/intel/llvm/blob/sycl/sycl/doc/GetStartedWithSYCLCompiler.md#prerequisites, please? If so, could you provide the logs from "configure" and "make" steps? |
|
Environment: Configure: Build build again to see errors: |
|
@crtrott can you please try this small hack? |
|
It build! |
@crtrott good to hear! Can you please tell me more about your Linux distro? What it's called and can I get it somewhere? I used this hack to build runtime on macOS, so I'm wondering why you need it on Linux too. |
|
This is more or less vanilla RedHat 6. But we install additional modules (using the 'module' package management system) which adds newer compilers and everything else. |
Strange... aligned_alloc should have worked. Anyway, I'm happy it is finally built. |
|
I think I had to force the global namespace once for something like this:
::aligned_alloc.
…On Wed, Sep 18, 2019 at 11:23 AM Alexander Batashev < ***@***.***> wrote:
This is more or less vanilla RedHat 6. But we install additional modules
(using the 'module' package management system) which adds newer compilers
and everything else.
Strange... aligned_alloc should have worked. Anyway, I'm happy it is
finally built.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#646?email_source=notifications&email_token=AAOWWASEPGXEOUUCKPJ3E23QKJIZRA5CNFSM4IXRKBX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7AUR2Y#issuecomment-532760811>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAOWWAVH2WJ433K7AOID7TTQKJIZRANCNFSM4IXRKBXQ>
.
|
|
It looks like aligned_alloc in C++ is C++17 feature. |
|
Yes aligned_alloc is part of C++17 (which sucks in C11). I didn't try to compile the compiler with GCC 9 though, so I didn't use a compiler which would enable C++17. I mean you could change your requirements to ask for C++17 compiler - it is not completely unreasonable to do so. But then you should also enforce C++17 in your CMakeList. |
I use exactly the same GCC version as you on daily basis and I see no problems. But my machine runs Ubuntu 18.04. Buildbot machines use GCC 7.4.0. Which is also far away from 9. I've also had positive experience with GCC 5.5.0. This is why that fail looks strange to me. |
|
std::aligned_malloc is c++17, ::aligned_malloc is c11
…On Wed, Sep 18, 2019 at 12:30 PM Christian Trott ***@***.***> wrote:
Yes aligned_alloc is part of C++17 (which sucks in C11). I didn't try to
compile the compiler with GCC 9 though, so I didn't use a compiler which
would enable C++17. I mean you could change your requirements to ask for
C++17 compiler - it is not completely unreasonable to do so. But then you
should also enforce C++17 in your CMakeList.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#646?email_source=notifications&email_token=AAOWWAUHRDZ5HDKPTUZRHTLQKJQUXA5CNFSM4IXRKBX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7A2ZCY#issuecomment-532786315>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAOWWAQYIUAB7EWUM464JRDQKJQUXANCNFSM4IXRKBXQ>
.
|
|
@alexbatashev, @crtrott, what is the status of this issue? Was the problem finally resolved? Do we need to submit any additional patches? Can we close this issue? |
|
@AlexeySachkov I believe this PR has fixed the issue: #763 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Following the instruction to build the compiler for x86 I am running into this compilation error:
Configure:
The text was updated successfully, but these errors were encountered: