On Tue, Apr 30, 2019 at 03:48:02PM -0600, Jeff Law wrote:
> On 4/30/19 11:24 AM, Matthew Malcomson wrote:
> > That was why I ended up suggesting multiple notes -- it's currently
> > trying to satisfy more than one criteria and they're not quite compatible.
> Well, we obviously have to keep arg setup, asan, stack protector and
> nonlocal stuff in the same relative order, but I believe they should all
> ultimately land before the NOTE_INSN_FUNCTION_BEG. THe question is how
> to make that happen :-)
The current meaning of NOTE_INSN_FUNCTION_BEG is
/* Indicate the beginning of the function body,
as opposed to parm setup. */
emit_note (NOTE_INSN_FUNCTION_BEG);
(function.c), and half of the things that use the note think that
everything before it is argument setup, and nothing after it is.
Just adding extra notes isn't enough afaics; some surgery is needed.
Segher
