Dne 28. 04. 25 v 10:58 Jakub Jelinek napsal(a):
On Sun, Apr 27, 2025 at 11:56:21AM +0200, Josef Melcr wrote:
* builtin-attrs.def (0): New int list.
This is just weird. Write
* builtin-attrs.def: Add DEF_LIST_INT_INT (0,2).
?
+DEF_CALLBACK_ATTRIBUTE(GOMP, 1, 0
this sort of thing, given their use case.
Thanks,
Josef Melcr
On 4/27/25 19:36, Andrew Pinski wrote:
On Sun, Apr 27, 2025 at 2:58 AM Josef Melcr wrote:
This patch enables constant propagation to outlined OpenMP kernels and
improves support for optimizing callback functions in general. It
st(int c) {
#pragma omp parallel for
for (int i = 0; i < c; i++) {
if (!__builtin_constant_p(c)) {
__builtin_abort();
}
a[i] = i;
}
}
int main() {
test(100);
return a[5] - 5;
}
With this patch, the body function will get cloned and the constant 'c'
will get propagated.
Bootstrapped a
ion is copied over, but I don't think that's the
case when using this function on its own. When a jump function is
constructed through different means, will the contents of the jump
functions still match ?
Best regards,
Josef Melcr
including atomic loads and stores. I am kind of at a loss for why that's
happening, I was hoping maybe somebody here could provide some insight ?
Thanks in advance :)
Best regards,
Josef Melcr
Dne 25. 10. 24 v 7:19 Josef Melcr napsal(a):
So I experimented a little and ran the testsuite a f
pping through. I would need to dig deeper to see which statements
those are. I am not quite sure where that leaves this patch.
Best regards,
Josef Melcr
Dne 24. 10. 24 v 19:45 Josef Melcr napsal(a):
Capital Remove
The second line should be just tab indented, not tab + 2 spaces, and
finished wit
llel statements. I'll try these changes and retest.
Best regards,
Josef Melcr
Dne 24. 10. 24 v 18:49 Jakub Jelinek napsal(a):
On Thu, Oct 24, 2024 at 04:37:24PM +0200, Josef Melcr wrote:
This patch removes a dead if statement checking for gomp-parallel gimple
statements. This if is in t
K for master ?
gcc/ChangeLog:
* cgraphbuild.cc (pass_build_cgraph_edges::execute): remove if
statement checking for gomp-parallel statements
Signed-off-by: Josef Melcr
---
gcc/cgraphbuild.cc | 6 --
1 file changed, 6 deletions(-)
diff --git a/gcc/cgraphbuild.cc
ChangeLog:
* MAINTAINERS: Add myself to write after approval
Signed-off-by: Josef Melcr
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 4a53521d8eb..f94aa9aeb79 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -657,6 +657,7 @@ Bryce
