You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "Wes McKinney (JIRA)" <ji...@apache.org> on 2019/08/13 13:38:00 UTC

[jira] [Updated] (ARROW-6205) [C++] ARROW_DEPRECATED warning when including io/interfaces.h from CUDA (.cu) source

     [ https://issues.apache.org/jira/browse/ARROW-6205?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Wes McKinney updated ARROW-6205:
--------------------------------
    Summary: [C++] ARROW_DEPRECATED warning when including io/interfaces.h from CUDA (.cu) source  (was: ARROW_DEPRECATED warning when including io/interfaces.h from CUDA (.cu) source)

> [C++] ARROW_DEPRECATED warning when including io/interfaces.h from CUDA (.cu) source
> ------------------------------------------------------------------------------------
>
>                 Key: ARROW-6205
>                 URL: https://issues.apache.org/jira/browse/ARROW-6205
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: C++
>    Affects Versions: 0.14.1
>            Reporter: Mark Harris
>            Assignee: Mark Harris
>            Priority: Minor
>              Labels: easyfix, pull-request-available
>             Fix For: 0.15.0
>
>          Time Spent: 2h
>  Remaining Estimate: 0h
>
> When including arrow/io/interfaces.h from a .cu source file (compiled with nvcc, the CUDA compiler), we get warnings like the following:
> {{[6/58] Building CUDA obje.../csv/csv_reader_impl.cu.}}
> {{arrow/install/include/arrow/io/interfaces.h(195): warning: attribute does not apply to any entity}}
> This example is from compiling [libcudf|[https://github.com/rapidsai/cudf].] libcudf installs these headers and includes them. This is a problem for libraries like libcudf which treat warnings as errors.
> There is a simple fix (I will submit a PR): change this code in interfaces.h:
> {{// TODO(kszucs): remove this after 0.13}}
> {{#ifndef _MSC_VER}}
> {{using WriteableFile ARROW_DEPRECATED("Use WritableFile") = WritableFile;}}
> {{using ReadableFileInterface ARROW_DEPRECATED("Use RandomAccessFile") = RandomAccessFile;}}
> {{#else}}{{// MSVC does not like using ARROW_DEPRECATED with using declarations}}{{using WriteableFile = WritableFile;}}
> {{using ReadableFileInterface = RandomAccessFile;}}
> {{#endif}}
>  
> Just change the ifndef to:
> {{#if not defined(_MSC_VER) && not defined(__CUDACC__)}}
> This fix should have no impact on other compilers.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)