This is an automated email from the ASF dual-hosted git repository.
fsaintjacques pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/master by this push:
new e9d1b8c ARROW-8592: [C++] Update docs to reflect LLVM 8
e9d1b8c is described below
commit e9d1b8c6aebb2ad3d63572b6a9649bfdfb2a99b7
Author: Micah Kornfield <[email protected]>
AuthorDate: Thu Apr 30 14:50:12 2020 -0400
ARROW-8592: [C++] Update docs to reflect LLVM 8
I'm not sure about the windows build instructions, if someone know for sure
it would be helpful.
Closes #7068 from emkornfield/ARROW-8592
Authored-by: Micah Kornfield <[email protected]>
Signed-off-by: François Saint-Jacques <[email protected]>
---
docs/source/developers/cpp/building.rst | 2 +-
docs/source/developers/cpp/development.rst | 2 +-
docs/source/developers/cpp/windows.rst | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/docs/source/developers/cpp/building.rst
b/docs/source/developers/cpp/building.rst
index 4d0a370..6559503 100644
--- a/docs/source/developers/cpp/building.rst
+++ b/docs/source/developers/cpp/building.rst
@@ -237,7 +237,7 @@ CMake:
LLVM and Clang Tools
~~~~~~~~~~~~~~~~~~~~
-We are currently using LLVM 7 for library builds and for other developer tools
+We are currently using LLVM 8 for library builds and for other developer tools
such as code formatting with ``clang-format``. LLVM can be installed via most
modern package managers (apt, yum, conda, Homebrew, chocolatey).
diff --git a/docs/source/developers/cpp/development.rst
b/docs/source/developers/cpp/development.rst
index b1e2d08..680f000 100644
--- a/docs/source/developers/cpp/development.rst
+++ b/docs/source/developers/cpp/development.rst
@@ -99,7 +99,7 @@ following checks:
In order to account for variations in the behavior of ``clang-format`` between
major versions of LLVM, we pin the version of ``clang-format`` used (current
-LLVM 7).
+LLVM 8).
Depending on how you installed clang-format, the build system may not be able
to find it. You can provide an explicit path to your LLVM installation (or the
diff --git a/docs/source/developers/cpp/windows.rst
b/docs/source/developers/cpp/windows.rst
index e86e77f..cdb6c19 100644
--- a/docs/source/developers/cpp/windows.rst
+++ b/docs/source/developers/cpp/windows.rst
@@ -307,7 +307,7 @@ tests can be made with there individual make targets).
SET APPVEYOR_BUILD_WORKER_IMAGE=Visual Studio 2017
SET USE_CLCACHE=false
SET ARROW_BUILD_GANDIVA=OFF
- SET ARROW_LLVM_VERSION=7.0.*
+ SET ARROW_LLVM_VERSION=8.0.*
SET PYTHON=3.6
SET ARCH=64
SET
PATH=C:\Users\User\Anaconda3;C:\Users\User\Anaconda3\Scripts;C:\Users\User\Anaconda3\Library\bin;%PATH%