On Fri, Mar 29, 2024 at 04:06:58PM +, Qing Zhao wrote:
> This is the 8th version of the patch.
Thanks for the updated version!
I've done a full Linux kernel build and run through my behavioral
regression test suite. Everything is working as expected.
-Kees
--
Kees Cook
> So, let’s delay the possible support to gdb in a later patch.
> Does this sound reasonable to you?
It's not really up to me, but sure. I was just curious if it perhaps
already worked, but not enough to apply the patches and find out.
Tom
Kees> Does DWARF have such an annotation? Regardless, I think this could be a
Kees> future patch to not hold up landing the initial feature.
Sure, the compiler can emit the array length (and structure size) as a
DWARF expression using the length.
Tom
Hi, Tom,
Thanks a lot for the comments.
It’s good to hear that this new attribute might be able to be used to help gdb.
We might spend some time to study to use this information in other consumers,
for example, gdb, in the future, if necessary and possible. If you have good
examples to
On Fri, Mar 29, 2024 at 12:09:15PM -0600, Tom Tromey wrote:
> > Qing Zhao writes:
>
> > This is the 8th version of the patch.
>
> > compare with the 7th version, the difference are:
>
> [...]
>
> Hi. I was curious to know if the information supplied by this attribute
> shows up in the
> Qing Zhao writes:
> This is the 8th version of the patch.
> compare with the 7th version, the difference are:
[...]
Hi. I was curious to know if the information supplied by this attribute
shows up in the DWARF. It would be good if it did, because that would
let gdb correctly print
Hi,
This is the 8th version of the patch.
compare with the 7th version, the difference are:
updates per Joseph's comments:
1. Wording changes in diagnostics;
"non flexible" to "non-flexible";
Diagnostics starts with a lowercase letter;
2. Documentation changes:
"named ``@var{count}''
