-
Notifications
You must be signed in to change notification settings - Fork 264
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
Update build-from-source instruction #515
Comments
Thanks for reporting, yes please, at https://github.com/root-project/root |
This is still not fixed, and it's a single line in the readme. |
Feel free to transfer this PR (#518) or (#523) to the upstream repo: https://github.com/root-project/root/pulls |
I cant build cling from source ; download the clingv1.0 release source code from github release ; download root project's llvm code in branch cling-latest ; Building CXX object tools/cling/lib/Interpreter/....clingInterpreter.dir/DynamicLibraryManagerSymbol.cpp.o
FAILED: tools/cling/lib/Interpreter/CMakeFiles/obj.clingInterpreter.dir/DynamicLibraryManagerSymbol.cpp.o
/usr/bin/clang++-14 -DCLING_VERSION=1.0 -DGTEST_HAS_RTTI=0 -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -I/home/hyj/code/cling-ninja/tools/cling/lib/Interpreter -I/home/hyj/code/cling-1.0/lib/Interpreter -I/home/hyj/code/cling-1.0/include -I/home/hyj/code/cling-ninja/tools/cling/include -isystem /home/hyj/code/cling-ninja/tools/clang/include -isystem /home/hyj/code/llvm-project/llvm/../clang/include -isystem /home/hyj/code/cling-ninja/include -isystem /home/hyj/code/llvm-project/llvm/include -fPIC -fno-semantic-interposition -fvisibility-inlines-hidden -Werror=date-time -Werror=unguarded-availability-new -Wall -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wmissing-field-initializers -pedantic -Wno-long-long -Wc++98-compat-extra-semi -Wimplicit-fallthrough -Wcovered-switch-default -Wno-noexcept-type -Wnon-virtual-dtor -Wdelete-non-virtual-dtor -Wsuggest-override -Wstring-conversion -Wmisleading-indentation -Wctad-maybe-unsupported -fdiagnostics-color -ffunction-sections -fdata-sections -fno-common -Woverloaded-virtual -Wno-nested-anon-types -Wno-covered-switch-default -Wno-unused-local-typedef -fno-rtti -O3 -DNDEBUG -std=c++17 -fvisibility=default -fvisibility-inlines-hidden -fno-exceptions -fno-rtti -MD -MT tools/cling/lib/Interpreter/CMakeFiles/obj.clingInterpreter.dir/DynamicLibraryManagerSymbol.cpp.o -MF tools/cling/lib/Interpreter/CMakeFiles/obj.clingInterpreter.dir/DynamicLibraryManagerSymbol.cpp.o.d -o tools/cling/lib/Interpreter/CMakeFiles/obj.clingInterpreter.dir/DynamicLibraryManagerSymbol.cpp.o -c /home/hyj/code/cling-1.0/lib/Interpreter/DynamicLibraryManagerSymbol.cpp
/home/hyj/code/cling-1.0/lib/Interpreter/DynamicLibraryManagerSymbol.cpp:387:16: error: 'set_size' is a private member of 'llvm::SmallVectorImpl<char>'
result.set_size(old_size);
^
/home/hyj/code/llvm-project/llvm/include/llvm/ADT/SmallVector.h:621:21: note: declared private here
using SuperClass::set_size; as you can see when i compile clingv1.0 , it fail; I also try the cling source code which release as January ;and use llvm in branch cling-llvm16 with a commit which was release as January too. |
Try maybe with cling git master branch |
Hobbyist who went down 🐇🕳️. https://github.com/Arcnor/cling/tree/8c65f199f1b116b4d33186886b162b36a3968d97 |
Explain what you would like to see improved
Update Building from source section of README.
LLVM has been updated
Optional: share how it could be improved
nvptx
should beNVPTX
. Build targets are case-sensitive. See https://llvm.org/docs/CMake.htmlCMAKE_BUILD_TYPE
is mandatory.To Reproduce
Copy and paste current README
First error
Second error
(After providing
CMAKE_BUILD_TYPE
)It is hard to resolve this error only with the diagnostic message since the real reason is that target is case-sensitive, not because
nvptx
is experimental.Setup
Additional context
Should I open a PR?
The text was updated successfully, but these errors were encountered: