[clang] [Clang] Fix 'counted_by' for nested struct pointers (PR #110437)

2024-09-30 Thread Jan Hendrik Farr via cfe-commits
Cydox wrote: Closing in favor of #110487 https://github.com/llvm/llvm-project/pull/110437 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Fix 'counted_by' for nested struct pointers (PR #110437)

2024-09-30 Thread Jan Hendrik Farr via cfe-commits
https://github.com/Cydox closed https://github.com/llvm/llvm-project/pull/110437 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Fix 'counted_by' for nested struct pointers (PR #110437)

2024-09-30 Thread Bill Wendling via cfe-commits
bwendling wrote: Thanks for this, but I think the issue is more basic. We should be issuing a "load" instruction after the `EmitMemberExpr` call. (That's what happens outside of using this in `__builtin_dynamic_object_size`.) https://github.com/llvm/llvm-project/pull/110437 ___

[clang] [Clang] Fix 'counted_by' for nested struct pointers (PR #110437)

2024-09-29 Thread Jan Hendrik Farr via cfe-commits
https://github.com/Cydox updated https://github.com/llvm/llvm-project/pull/110437 >From e1ef2156ed4a0f3ccd1d6dca97faa014284e4605 Mon Sep 17 00:00:00 2001 From: Jan Hendrik Farr Date: Sun, 29 Sep 2024 21:38:13 +0200 Subject: [PATCH] [Clang] Fix 'counted_by' for nested struct pointers Fixes #110

[clang] [Clang] Fix 'counted_by' for nested struct pointers (PR #110437)

2024-09-29 Thread Jan Hendrik Farr via cfe-commits
Cydox wrote: @bwendling please review https://github.com/llvm/llvm-project/pull/110437 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Fix 'counted_by' for nested struct pointers (PR #110437)

2024-09-29 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-codegen Author: Jan Hendrik Farr (Cydox) Changes Fixes #110385 Fix counted_by attribute for cases where the flexible array member is accessed through struct pointer inside another struct: struct variable { int a; int b; i

[clang] [Clang] Fix 'counted_by' for nested struct pointers (PR #110437)

2024-09-29 Thread via cfe-commits
github-actions[bot] wrote: Thank you for submitting a Pull Request (PR) to the LLVM Project! This PR will be automatically labeled and the relevant teams will be notified. If you wish to, you can add reviewers by using the "Reviewers" section on this page. If this is not working for you, it

[clang] [Clang] Fix 'counted_by' for nested struct pointers (PR #110437)

2024-09-29 Thread Jan Hendrik Farr via cfe-commits
https://github.com/Cydox created https://github.com/llvm/llvm-project/pull/110437 Fixes #110385 Fix counted_by attribute for cases where the flexible array member is accessed through struct pointer inside another struct: struct variable { int a; int b; int length;