Follow-up to r353959 and r368070: do the same for other architectures.
arm32 already seems to use its own .fnstart/.fnend directives, which
appear to be ARM-specific variants of the same thing. Likewise, MIPS
uses .frame directives.
Paths
| Differential D27387 Authored by cem on Nov 26 2020, 6:27 PM.
Details Summary Follow-up to r353959 and r368070: do the same for other architectures. arm32 already seems to use its own .fnstart/.fnend directives, which Test Plan Tinderbox running.
Diff Detail
Event TimelineHerald added a reviewer: manu. · View Herald TranscriptNov 26 2020, 6:27 PM2020-11-26 18:27:51 (UTC+0) Harbormaster completed remote builds in B35057: Diff 80034.Nov 26 2020, 6:27 PM2020-11-26 18:27:54 (UTC+0) jrtc27 added inline comments. This revision now requires changes to proceed.Nov 26 2020, 6:35 PM2020-11-26 18:35:16 (UTC+0) cem marked an inline comment as done. Comment Actions
Harbormaster completed remote builds in B35058: Diff 80035.Nov 26 2020, 7:08 PM2020-11-26 19:08:33 (UTC+0)
Comment Actions Add a lot of missing END directives. Tinderbox passes on arm64 and i386. Herald added a reviewer: jmg. · View Herald TranscriptNov 26 2020, 9:11 PM2020-11-26 21:11:56 (UTC+0) Harbormaster completed remote builds in B35063: Diff 80041.Nov 26 2020, 9:11 PM2020-11-26 21:11:57 (UTC+0) cem marked 2 inline comments as done. Harbormaster completed remote builds in B35066: Diff 80044.Nov 26 2020, 10:34 PM2020-11-26 22:34:18 (UTC+0) Harbormaster completed remote builds in B35135: Diff 80174.Dec 1 2020, 8:03 AM2020-12-01 08:03:13 (UTC+0) Comment Actions Are there any outstanding objections? (I think I've addressed everything raised so far, but please correct me if I am mistaken.) Is there anyone who hasn't reviewed yet, but will have time to review in the next few days? Thanks. Comment Actions Looks good to me. Obviously there might be some non-default build configurations that are missing end() calls, but if so they been be fixed later. Probably best to wait for at least one more review though. This revision was not accepted when it landed; it landed in state Needs Review.Dec 5 2020, 12:34 AM2020-12-05 00:34:04 (UTC+0) Closed by commit rS368354: Add CFI start/end proc directives to arm64, i386, and ppc (authored by cem). · Explain Why This revision was automatically updated to reflect the committed changes. Comment Actions I'm fine with it on readthrough, I just hadn't gotten around to the actual tests. If it broke anything in powerpc land, it should be easy to fix. Thanks for tidying up the procedure annotations! Comment Actions
Thanks! It compiles (tinderbox, anyway) and shouldn't affect code generation, so I'm cautiously optimistic nothing broke. Please let me know if it broke something and I can help fix it :-).
Revision Contents
Diff 80347 head/lib/libc/powerpc/SYS.h
head/lib/libc/powerpc64/SYS.h
head/lib/libc/powerpc64/sys/cerror.S
head/lib/libthr/arch/i386/i386/_umtx_op_err.S
|
Since you're touching that line, use https: everywhere for consistency. (I checked and all appear to work.)