[PATCH] D38770: AMDGPU: Use stricter bounds for workitem builtins

2017-10-16 Thread Tony Tye via Phabricator via cfe-commits
t-tye added inline comments. Comment at: include/clang/Basic/TargetInfo.h:1060 + /// \returns Maximum device supported OpenCL workgroup size. + virtual unsigned getOpenCLMaxWorkGroupSize(unsigned Dim) const { +return 0; Is this specifically tied to OpenCL o

[PATCH] D38770: AMDGPU: Use stricter bounds for workitem builtins

2017-10-16 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm updated this revision to Diff 119209. arsenm added a comment. Use 1024 for OpenCL https://reviews.llvm.org/D38770 Files: include/clang/Basic/TargetInfo.h lib/Basic/Targets/AMDGPU.cpp lib/Basic/Targets/AMDGPU.h lib/CodeGen/CGBuiltin.cpp test/CodeGenOpenCL/builtins-amdgcn.cl te

[PATCH] D38770: AMDGPU: Use stricter bounds for workitem builtins

2017-10-11 Thread Nicolai Hähnle via Phabricator via cfe-commits
nhaehnle accepted this revision. nhaehnle added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D38770 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/c

[PATCH] D38770: AMDGPU: Use stricter bounds for workitem builtins

2017-10-10 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm created this revision. Herald added subscribers: t-tye, tpr, dstuttard, nhaehnle, wdng. https://reviews.llvm.org/D38770 Files: include/clang/Basic/TargetInfo.h lib/Basic/Targets/AMDGPU.cpp lib/Basic/Targets/AMDGPU.h lib/CodeGen/CGBuiltin.cpp test/CodeGenOpenCL/builtins-amdgcn.cl