r363376 - Revert "[OpenCL] Move OpenCLBuiltins.td and remove unused include"

Nico Weber via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 14 05:26:24 PDT 2019


Most builds seem to be happy. Since you're seeing this problem, maybe you
could debug it a bit?

On Fri, Jun 14, 2019 at 8:11 AM Sven van Haastregt via cfe-commits <
cfe-commits at lists.llvm.org> wrote:

> Author: svenvh
> Date: Fri Jun 14 05:14:58 2019
> New Revision: 363376
>
> URL: http://llvm.org/viewvc/llvm-project?rev=363376&view=rev
> Log:
> Revert "[OpenCL] Move OpenCLBuiltins.td and remove unused include"
>
> This reverts commit r363242 as it broke some builds with
>
>   make[2]: *** No rule to make target 'ClangOpenCLBuiltinsImpl', needed by
>   'tools/clang/lib/Sema/CMakeFiles/obj.clangSema.dir/SemaLookup.cpp.o'.
>
> Added:
>     cfe/trunk/include/clang/Basic/OpenCLBuiltins.td
>       - copied, changed from r363369, cfe/trunk/lib/Sema/OpenCLBuiltins.td
> Removed:
>     cfe/trunk/lib/Sema/OpenCLBuiltins.td
> Modified:
>     cfe/trunk/include/clang/Basic/CMakeLists.txt
>     cfe/trunk/lib/Sema/CMakeLists.txt
>     cfe/trunk/lib/Sema/SemaLookup.cpp
>
> Modified: cfe/trunk/include/clang/Basic/CMakeLists.txt
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/CMakeLists.txt?rev=363376&r1=363375&r2=363376&view=diff
>
> ==============================================================================
> --- cfe/trunk/include/clang/Basic/CMakeLists.txt (original)
> +++ cfe/trunk/include/clang/Basic/CMakeLists.txt Fri Jun 14 05:14:58 2019
> @@ -41,6 +41,12 @@ clang_tablegen(AttrHasAttributeImpl.inc
>    TARGET ClangAttrHasAttributeImpl
>    )
>
> +clang_tablegen(OpenCLBuiltins.inc
> +  -I ${CMAKE_CURRENT_SOURCE_DIR}/../../ -gen-clang-opencl-builtins
> +  SOURCE OpenCLBuiltins.td
> +  TARGET ClangOpenCLBuiltinsImpl
> +  )
> +
>  # ARM NEON
>  clang_tablegen(arm_neon.inc -gen-arm-neon-sema
>    SOURCE arm_neon.td
>
> Copied: cfe/trunk/include/clang/Basic/OpenCLBuiltins.td (from r363369,
> cfe/trunk/lib/Sema/OpenCLBuiltins.td)
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/OpenCLBuiltins.td?p2=cfe/trunk/include/clang/Basic/OpenCLBuiltins.td&p1=cfe/trunk/lib/Sema/OpenCLBuiltins.td&r1=363369&r2=363376&rev=363376&view=diff
>
> ==============================================================================
>     (empty)
>
> Modified: cfe/trunk/lib/Sema/CMakeLists.txt
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/CMakeLists.txt?rev=363376&r1=363375&r2=363376&view=diff
>
> ==============================================================================
> --- cfe/trunk/lib/Sema/CMakeLists.txt (original)
> +++ cfe/trunk/lib/Sema/CMakeLists.txt Fri Jun 14 05:14:58 2019
> @@ -66,10 +66,3 @@ add_clang_library(clangSema
>    clangEdit
>    clangLex
>    )
> -
> -clang_tablegen(OpenCLBuiltins.inc -gen-clang-opencl-builtins
> -  SOURCE OpenCLBuiltins.td
> -  TARGET ClangOpenCLBuiltinsImpl
> -  )
> -
> -set_source_files_properties(SemaLookup.cpp OBJECT_DEPENDS
> ClangOpenCLBuiltinsImpl)
>
> Removed: cfe/trunk/lib/Sema/OpenCLBuiltins.td
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/OpenCLBuiltins.td?rev=363375&view=auto
>
> ==============================================================================
> --- cfe/trunk/lib/Sema/OpenCLBuiltins.td (original)
> +++ cfe/trunk/lib/Sema/OpenCLBuiltins.td (removed)
> @@ -1,296 +0,0 @@
> -//==--- OpenCLBuiltins.td - OpenCL builtin declarations
> -------------------===//
> -//
> -//                     The LLVM Compiler Infrastructure
> -//
> -// Part of the LLVM Project, under the Apache License v2.0 with LLVM
> Exceptions.
> -// See https://llvm.org/LICENSE.txt for license information.
> -// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
> -//
>
> -//===----------------------------------------------------------------------===//
> -//
> -// This file contains TableGen definitions for OpenCL builtin function
> -// declarations.  In case of an unresolved function name in OpenCL, Clang
> will
> -// check for a function described in this file when
> -fdeclare-opencl-builtins
> -// is specified.
> -//
>
> -//===----------------------------------------------------------------------===//
> -
>
> -//===----------------------------------------------------------------------===//
> -//              Definitions of miscellaneous basic entities.
>
> -//===----------------------------------------------------------------------===//
> -// Versions of OpenCL
> -class Version<int _Version> {
> -  int Version = _Version;
> -}
> -def CL10: Version<100>;
> -def CL11: Version<110>;
> -def CL12: Version<120>;
> -def CL20: Version<200>;
> -
> -// Address spaces
> -// Pointer types need to be assigned an address space.
> -class AddressSpace<string _AS> {
> -  string AddrSpace = _AS;
> -}
> -def default_as    : AddressSpace<"clang::LangAS::Default">;
> -def private_as    : AddressSpace<"clang::LangAS::opencl_private">;
> -def global_as     : AddressSpace<"clang::LangAS::opencl_global">;
> -def constant_as   : AddressSpace<"clang::LangAS::opencl_constant">;
> -def local_as      : AddressSpace<"clang::LangAS::opencl_local">;
> -def generic_as    : AddressSpace<"clang::LangAS::opencl_generic">;
> -
> -
> -// Qualified Type. Allow to retrieve one ASTContext QualType.
> -class QualType<string _Name> {
> -  // Name of the field or function in a clang::ASTContext
> -  // E.g. Name="IntTy" for the int type, and "getIntPtrType()" for an
> intptr_t
> -  string Name = _Name;
> -}
> -
> -// Helper class to store type access qualifiers (volatile, const, ...).
> -class Qualifier<string _QualName> {
> -  string QualName = _QualName;
> -}
> -
>
> -//===----------------------------------------------------------------------===//
> -//                      OpenCL C classes for types
>
> -//===----------------------------------------------------------------------===//
> -// OpenCL types (int, float, ...)
> -class Type<string _Name, QualType _QTName> {
> -  // Name of the Type
> -  string Name = _Name;
> -  // QualType associated with this type
> -  QualType QTName = _QTName;
> -  // Size of the vector (if applicable)
> -  int VecWidth = 0;
> -  // Is pointer
> -  bit IsPointer = 0;
> -  // List of qualifiers associated with the type (volatile, ...)
> -  list<Qualifier> QualList = [];
> -  // Address space
> -  string AddrSpace = "clang::LangAS::Default";
> -  // Access qualifier. Must be one of ("RO", "WO", "RW").
> -  string AccessQualifier = "";
> -}
> -
> -// OpenCL vector types (e.g. int2, int3, int16, float8, ...)
> -class VectorType<Type _Ty, int _VecWidth> : Type<_Ty.Name, _Ty.QTName> {
> -  int VecWidth = _VecWidth;
> -}
> -
> -// OpenCL pointer types (e.g. int*, float*, ...)
> -class PointerType<Type _Ty, AddressSpace _AS = global_as> :
> -                                      Type<_Ty.Name, _Ty.QTName> {
> -  bit IsPointer = 1;
> -  string AddrSpace = _AS.AddrSpace;
> -}
> -
> -// OpenCL image types (e.g. image2d_t, ...)
> -class ImageType<Type _Ty, QualType _QTName, string _AccessQualifier> :
> -                                              Type<_Ty.Name, _QTName> {
> -  let AccessQualifier = _AccessQualifier;
> -}
> -
>
> -//===----------------------------------------------------------------------===//
> -//                      OpenCL C class for builtin functions
>
> -//===----------------------------------------------------------------------===//
> -class Builtin<string _Name, list<Type> _Signature> {
> -  // Name of the builtin function
> -  string Name = _Name;
> -  // List of types used by the function. The first one is the return type
> and
> -  // the following are the arguments. The list must have at least one
> element
> -  // (the return type).
> -  list<Type> Signature = _Signature;
> -  // OpenCL Extension to which the function belongs (cl_khr_subgroups,
> ...)
> -  string Extension = "";
> -  // OpenCL Version to which the function belongs (CL10, ...)
> -  Version Version = CL10;
> -}
> -
>
> -//===----------------------------------------------------------------------===//
> -//                           Multiclass definitions
>
> -//===----------------------------------------------------------------------===//
> -// multiclass BifN: Creates Builtin class instances for OpenCL builtin
> -//                  functions with N arguments.
> -// _Name      : Name of the function
> -// _Signature : Signature of the function (list of the Type used by the
> -//              function, the first one being the return type).
> -// _IsVector  : List of bit indicating if the type in the _Signature at
> the
> -//              same index is to be a vector in the multiple overloads.
> The
> -//              list must have at least one non-zero value.
> -multiclass Bif0<string _Name, list<Type> _Signature, list<bit> _IsVector>
> {
> -  def : Builtin<_Name, _Signature>;
> -  foreach v = [2, 3, 4, 8, 16] in {
> -    def : Builtin<_Name,
> -                  [!if(_IsVector[0], VectorType<_Signature[0], v>,
> _Signature[0])]>;
> -  }
> -}
> -multiclass Bif1<string _Name, list<Type> _Signature, list<bit> _IsVector>
> {
> -  def : Builtin<_Name, _Signature>;
> -  foreach v = [2, 3, 4, 8, 16] in {
> -    def : Builtin<_Name,
> -                  [!if(_IsVector[0], VectorType<_Signature[0], v>,
> _Signature[0]),
> -                  !if(_IsVector[1], VectorType<_Signature[1], v>,
> _Signature[1])]>;
> -  }
> -}
> -multiclass Bif2<string _Name, list<Type> _Signature, list<bit> _IsVector>
> {
> -  def : Builtin<_Name, _Signature>;
> -  foreach v = [2, 3, 4, 8, 16] in {
> -    def : Builtin<_Name,
> -                  [!if(_IsVector[0], VectorType<_Signature[0], v>,
> _Signature[0]),
> -                  !if(_IsVector[1], VectorType<_Signature[1], v>,
> _Signature[1]),
> -                  !if(_IsVector[2], VectorType<_Signature[2], v>,
> _Signature[2])]>;
> -  }
> -}
> -multiclass Bif3<string _Name, list<Type> _Signature, list<bit> _IsVector>
> {
> -  def : Builtin<_Name, _Signature>;
> -  foreach v = [2, 3, 4, 8, 16] in {
> -    def : Builtin<_Name,
> -                  [!if(_IsVector[0], VectorType<_Signature[0], v>,
> _Signature[0]),
> -                  !if(_IsVector[1], VectorType<_Signature[1], v>,
> _Signature[1]),
> -                  !if(_IsVector[2], VectorType<_Signature[2], v>,
> _Signature[2]),
> -                  !if(_IsVector[3], VectorType<_Signature[3], v>,
> _Signature[3])]>;
> -  }
> -}
>
> -//===----------------------------------------------------------------------===//
> -//                 Definitions of OpenCL C types
>
> -//===----------------------------------------------------------------------===//
> -// OpenCL v1.2 s6.1.1: Built-in Scalar Data Types
> -def bool_t      : Type<"bool", QualType<"BoolTy">>;
> -def char_t      : Type<"char", QualType<"CharTy">>;
> -def uchar_t     : Type<"uchar", QualType<"UnsignedCharTy">>;
> -def short_t     : Type<"short", QualType<"ShortTy">>;
> -def ushort_t    : Type<"ushort", QualType<"UnsignedShortTy">>;
> -def int_t       : Type<"int", QualType<"IntTy">>;
> -def uint_t      : Type<"uint", QualType<"UnsignedIntTy">>;
> -def long_t      : Type<"long", QualType<"LongTy">>;
> -def ulong_t     : Type<"ulong", QualType<"UnsignedLongTy">>;
> -def float_t     : Type<"float", QualType<"FloatTy">>;
> -def double_t    : Type<"double", QualType<"DoubleTy">>;
> -def half_t      : Type<"half", QualType<"HalfTy">>;
> -def size_t      : Type<"size_t",  QualType<"getSizeType()">>;
> -def ptrdiff_t   : Type<"ptrdiff_t", QualType<"getPointerDiffType()">>;
> -def intptr_t    : Type<"intptr_t", QualType<"getIntPtrType()">>;
> -def uintptr_t   : Type<"uintptr_t", QualType<"getUIntPtrType()">>;
> -def void_t      : Type<"void", QualType<"VoidTy">>;
> -
> -// OpenCL v1.2 s6.1.2: Built-in Vector Data Types
> -foreach v = [2, 3, 4, 8, 16] in {
> -  def char#v#_t    : VectorType<char_t, v>;
> -  def uchar#v#_t   : VectorType<uchar_t, v>;
> -  def short#v#_t   : VectorType<short_t, v>;
> -  def ushort#v#_t  : VectorType<ushort_t, v>;
> -  def "int"#v#_t   : VectorType<int_t, v>;
> -  def uint#v#_t    : VectorType<uint_t, v>;
> -  def long#v#_t    : VectorType<long_t, v>;
> -  def ulong#v#_t   : VectorType<ulong_t, v>;
> -  def float#v#_t   : VectorType<float_t, v>;
> -  def double#v#_t  : VectorType<double_t, v>;
> -  def half#v#_t    : VectorType<half_t, v>;
> -}
> -
> -// OpenCL v1.2 s6.1.3: Other Built-in Data Types
> -// These definitions with a "null" name are "abstract". They should not
> -// be used in definitions of Builtin functions.
> -def image2d_t         : Type<"image2d_t", QualType<"null">>;
> -def image3d_t         : Type<"image3d_t", QualType<"null">>;
> -def image2d_array_t   : Type<"image2d_array_t", QualType<"null">>;
> -def image1d_t         : Type<"image1d_t", QualType<"null">>;
> -def image1d_buffer_t  : Type<"image1d_buffer_t", QualType<"null">>;
> -def image1d_array_t   : Type<"image1d_array_t", QualType<"null">>;
> -// Unlike the few functions above, the following definitions can be used
> -// in definitions of Builtin functions (they have a QualType with a name).
> -foreach v = ["RO", "WO", "RW"] in {
> -  def image2d_#v#_t       : ImageType<image2d_t,
> -                                      QualType<"OCLImage2d"#v#"Ty">,
> -                                      v>;
> -  def image3d_#v#_t       : ImageType<image3d_t,
> -                                      QualType<"OCLImage3d"#v#"Ty">,
> -                                      v>;
> -  def image2d_array#v#_t  : ImageType<image2d_array_t,
> -                                      QualType<"OCLImage2dArray"#v#"Ty">,
> -                                      v>;
> -  def image1d_#v#_t       : ImageType<image1d_t,
> -                                      QualType<"OCLImage1d"#v#"Ty">,
> -                                      v>;
> -  def image1d_buffer#v#_t : ImageType<image1d_buffer_t,
> -                                      QualType<"OCLImage1dBuffer"#v#"Ty">,
> -                                      v>;
> -  def image1d_array#v#_t  : ImageType<image1d_array_t,
> -                                      QualType<"OCLImage1dArray"#v#"Ty">,
> -                                      v>;
> -}
> -
> -def sampler_t         : Type<"sampler_t", QualType<"OCLSamplerTy">>;
> -def event_t           : Type<"event_t", QualType<"OCLEventTy">>;
> -
>
> -//===----------------------------------------------------------------------===//
> -//                 Definitions of OpenCL builtin functions
>
> -//===----------------------------------------------------------------------===//
> -// OpenCL v1.2 s6.2.3: Explicit Conversions
> -// Generate the convert_ builtins.
> -foreach RType = [float_t, double_t, char_t, uchar_t, short_t, ushort_t,
> -                int_t, uint_t, long_t, ulong_t] in {
> -  foreach IType = [float_t, double_t, char_t, uchar_t, short_t, ushort_t,
> -                   int_t, uint_t, long_t, ulong_t] in {
> -    foreach sat = ["", "_sat"] in {
> -      foreach rte = ["", "_rte", "_rtz", "_rtp", "_rtn"] in {
> -        def : Builtin<"convert_"  # RType.Name # sat # rte, [RType,
> IType]>;
> -        foreach v = [2, 3, 4, 8, 16] in {
> -          def : Builtin<"convert_" # RType.Name # v # sat # rte,
> -                        [VectorType<RType, v>,
> -                         VectorType<IType, v>]>;
> -        }
> -      }
> -    }
> -  }
> -}
> -
> -// OpenCL v1.2 s6.12.1: Work-Item Functions
> -def get_work_dim : Builtin<"get_work_dim", [uint_t]>;
> -foreach name = ["get_global_size", "get_global_id", "get_local_size",
> -                "get_local_id", "get_num_groups", "get_group_id",
> -                "get_global_offset"] in {
> -  def : Builtin<name, [size_t, uint_t]>;
> -}
> -
> -// OpenCL v1.2 s6.12.2: Math Functions
> -foreach name = ["acos", "acosh", "acospi",
> -                "asin", "asinh", "asinpi",
> -                "atan", "atanh", "atanpi"] in {
> -  foreach type = [float_t, double_t, half_t] in {
> -    defm : Bif1<name, [type, type], [1, 1]>;
> -  }
> -}
> -
> -foreach name = ["atan2", "atan2pi"] in {
> -  foreach type = [float_t, double_t, half_t] in {
> -    defm : Bif2<name, [type, type, type], [1, 1, 1]>;
> -  }
> -}
> -
> -foreach name = ["fmax", "fmin"] in {
> -  foreach type = [float_t, double_t, half_t] in {
> -    defm : Bif2<name, [type, type, type], [1, 1, 1]>;
> -    defm : Bif2<name, [type, type, type], [1, 1, 0]>;
> -  }
> -}
> -
> -// OpenCL v1.2 s6.12.14: Built-in Image Read Functions
> -def read_imagef : Builtin<"read_imagef",
> -                          [float4_t, image2d_RO_t, VectorType<int_t, 2>]>;
> -def write_imagef : Builtin<"write_imagef",
> -                           [void_t,
> -                            image2d_WO_t,
> -                            VectorType<int_t, 2>,
> -                            VectorType<float_t, 4>]>;
> -
> -
> -// OpenCL v2.0 s9.17.3: Additions to section 6.13.1: Work-Item Functions
> -let Version = CL20 in {
> -  let Extension = "cl_khr_subgroups" in {
> -    def get_sub_group_size : Builtin<"get_sub_group_size", [uint_t]>;
> -    def get_max_sub_group_size : Builtin<"get_max_sub_group_size",
> [uint_t]>;
> -    def get_num_sub_groups : Builtin<"get_num_sub_groups", [uint_t]>;
> -  }
> -}
>
> Modified: cfe/trunk/lib/Sema/SemaLookup.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaLookup.cpp?rev=363376&r1=363375&r2=363376&view=diff
>
> ==============================================================================
> --- cfe/trunk/lib/Sema/SemaLookup.cpp (original)
> +++ cfe/trunk/lib/Sema/SemaLookup.cpp Fri Jun 14 05:14:58 2019
> @@ -46,7 +46,7 @@
>  #include <utility>
>  #include <vector>
>
> -#include "OpenCLBuiltins.inc"
> +#include "clang/Basic/OpenCLBuiltins.inc"
>
>  using namespace clang;
>  using namespace sema;
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190614/972556a6/attachment-0001.html>


More information about the cfe-commits mailing list