Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

compile failed (OrcRemoteTargetClient.h:722:26: error: could not convert) #297

Closed
derofim opened this issue Aug 17, 2019 · 2 comments
Closed

Comments

@derofim
Copy link

derofim commented Aug 17, 2019

Issue: OrcRemoteTargetClient.h:722:26: error: could not convert (see below)

Please update https://root.cern.ch/cling-build-instructions if need to switch branches.

Ubuntu 18.04 LTS 
gcc version 8.2.0 (Ubuntu 8.2.0-7ubuntu1)

Similar issue: #279

Possible patches:

Build script:

#!/bin/bash

set +x
set -e

# see https://root.cern.ch/cling-build-instructions

# mkdir from some parent project dir!
mkdir -p ../cling-build
cd ../cling-build

git clone http://root.cern.ch/git/llvm.git src
cd src
git checkout cling-patches
cd tools
git clone http://root.cern.ch/git/cling.git
git clone http://root.cern.ch/git/clang.git
cd clang
git checkout cling-patches
cd ../../..

mkdir build
cd build

# add parallel build support
# https://stackoverflow.com/questions/601970/how-do-i-utilise-all-the-cores-for-nmake
set CL=/MP

cmake -DCMAKE_INSTALL_PREFIX=../release -DCMAKE_BUILD_TYPE=Release ../src
cmake --build . -- -j6

# no install
# cmake --build . --target install

# NOTE: if you get error
# /home/denis/workspace/cling-cmake/cling-build/src/include/llvm/ExecutionEngine/Orc/OrcRemoteTargetClient.h:722:26: error: could not convert ‘((llvm::orc::remote::OrcRemoteTargetClient<ChannelT>*)this)->callB<llvm::orc::remote::OrcRemoteTargetRPCAPI::ReadMem>(Src, Size)’ from ‘Expected<vector<unsigned char,allocator<unsigned char>>>’ to ‘Expected<vector<char,allocator<char>>>’
# return callB<ReadMem>(Src, Size);
# than apply patch (replace char with uint8_t) https://aur.archlinux.org/cgit/aur.git/tree/gcc-build-fix.patch?h=llvm50
@Axel-Naumann
Copy link
Member

Could you do me a big favor and check whether http://root.cern.ch/gitweb?p=cling.git;a=commitdiff;h=2c92b57062e22c14c015c7bd3f9c7d9d3197defa fixes this?

@Axel-Naumann
Copy link
Member

Confirmed fixed in #302 thus closing.

Axel-Naumann added a commit to Axel-Naumann/cling-all-in-one that referenced this issue Sep 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants