-
-
Notifications
You must be signed in to change notification settings - Fork 202
set static const values to static constexpr #2830
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
Changes from all commits
35e3127
dbe2429
ca0ca56
9c35e89
8d95623
f200221
dbf86a0
f6b5c52
b8f8a60
04c1c98
9f2af5a
e21588e
265f0cf
9af3659
ca927a1
0bc2be0
1d2d4aa
67d3781
9a12c77
fa56ba8
7fbffe0
184978f
88c4211
1a9b1c1
bef2174
7ea0296
8779172
7f2b34e
5ca9e19
1e503a4
12597ce
10514e1
a16df24
092c0be
e9cf9cf
b527a01
ef5fbdc
d2d4cee
b634978
5be6653
57cc140
87245ad
c7dd4fd
1cf8c04
952ea3a
f365972
e9987b6
44dd969
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -12,7 +12,7 @@ namespace math { | |||||
| namespace opencl_kernels { | ||||||
|
|
||||||
| // \cond | ||||||
| static const std::string add_batch_kernel_code = STRINGIFY( | ||||||
| static constexpr const char *add_batch_kernel_code = STRINGIFY( | ||||||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
For consistency |
||||||
| // \endcond | ||||||
| /** \ingroup opencl_kernels | ||||||
| * Sums a batch of matrices. Buffer A contains | ||||||
|
|
||||||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -10,7 +10,7 @@ namespace stan { | |||||
| namespace math { | ||||||
| namespace opencl_kernels { | ||||||
| // \cond | ||||||
| static const std::string is_symmetric_kernel_code = STRINGIFY( | ||||||
| static constexpr const char *is_symmetric_kernel_code = STRINGIFY( | ||||||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
| // \endcond | ||||||
| /** \ingroup opencl_kernels | ||||||
| * Check if the <code>matrix_cl</code> is symmetric | ||||||
|
|
||||||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -12,7 +12,7 @@ namespace math { | |||||
| namespace opencl_kernels { | ||||||
|
|
||||||
| // \cond | ||||||
| static const char *cumulative_sum1_kernel_code = STRINGIFY( | ||||||
| static constexpr const char *cumulative_sum1_kernel_code = STRINGIFY( | ||||||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
| // \endcond | ||||||
| /** \ingroup opencl_kernels | ||||||
| * First kernel of the cumulative sum implementation. Each thread sums the | ||||||
|
|
@@ -62,7 +62,7 @@ static const char *cumulative_sum1_kernel_code = STRINGIFY( | |||||
| // \endcond | ||||||
|
|
||||||
| // \cond | ||||||
| static const char *cumulative_sum2_kernel_code = STRINGIFY( | ||||||
| static constexpr const char *cumulative_sum2_kernel_code = STRINGIFY( | ||||||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
| // \endcond | ||||||
| /** \ingroup opencl_kernels | ||||||
| * Second kernel of the cumulative sum implementation. Calculates prefix sum | ||||||
|
|
@@ -116,7 +116,7 @@ static const char *cumulative_sum2_kernel_code = STRINGIFY( | |||||
| // \endcond | ||||||
|
|
||||||
| // \cond | ||||||
| static const char *cumulative_sum3_kernel_code = STRINGIFY( | ||||||
| static constexpr const char *cumulative_sum3_kernel_code = STRINGIFY( | ||||||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
| // \endcond | ||||||
| /** \ingroup opencl_kernels | ||||||
| * Third kernel of the cumulative sum implementation. Given sums of threads | ||||||
|
|
||||||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -10,7 +10,7 @@ namespace stan { | |||||
| namespace math { | ||||||
| namespace opencl_kernels { | ||||||
| // \cond | ||||||
| static const std::string divide_columns_kernel_code = STRINGIFY( | ||||||
| static constexpr const char *divide_columns_kernel_code = STRINGIFY( | ||||||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
| // \endcond | ||||||
| /** \ingroup opencl_kernels | ||||||
| * Takes vector A and divides columns vector in A element-wise by the values | ||||||
|
|
||||||
Uh oh!
There was an error while loading. Please reload this page.