https://sourceware.org/bugzilla/show_bug.cgi?id=32879
--- Comment #2 from Indu Bhagat <indu.bhagat at oracle dot com> --- As per the documentation, .cfi_def_cfa_register modifies a rule for computing CFA; the register is updated, but the offset remains the same. So I think the following suffices: if (last_fre) sframe_fre_set_cfa_offset (cur_fre, last_fre->cfa_offset); merge_candidate is being reset to false already. The value of merge_candidate is of importance when we deal with the advance_loc opcode. merge_candidate helps decide when the end boundary of the cur_fre is firm. As for the testcase, generally speaking, the common-empty-XX.d tests are for cases when the test are run on all applicable targets (hence "common") and when we skip FDE generation (hence "empty"). So for this patch, one option is to just run a test for one backend. I think thats OK. So we can add the new test as cfi-sframe-x86_64-2.s with a reference to the PR inside the cfi-sframe-x86_64-2.s. -- You are receiving this mail because: You are on the CC list for the bug.
