Bug 205 - .align directive
Summary: .align directive
Status: RESOLVED FIXED
Alias: None
Product: RMAC
Classification: Unclassified
Component: Core (show other bugs)
Version: unspecified
Hardware: All All
: Normal enhancement
Assignee: Shamus Hammons
URL:
Depends on:
Blocks:
 
Reported: 2022-07-23 12:12 CDT by ggn
Modified: 2022-08-16 12:23 CDT (History)
1 user (show)

See Also:


Attachments
Bastian's original patch (for posterity) (1.64 KB, patch)
2022-07-23 12:12 CDT, ggn
Details
My patch (including doc updates and some bits here and there) (12.37 KB, patch)
2022-07-23 12:14 CDT, ggn
Details
Second try (apparently some people define "align" as a macro) (12.36 KB, patch)
2022-07-25 07:45 CDT, ggn
Details
No idea what happened to the previous one. Absolutely none (11.46 KB, patch)
2022-08-16 01:27 CDT, ggn
Details

Note You need to log in before you can comment on or make changes to this bug.
Description ggn 2022-07-23 12:12:28 CDT
Bastian shared a patch for arbitrary alignment here: https://atariage.com/forums/topic/234634-rmacrln/?do=findComment&comment=5079657

I merely tested it under a few configurations and added proper 56001 handling. Looks fine to me!
Comment 1 ggn 2022-07-23 12:12:53 CDT
Created attachment 187 [details]
Bastian's original patch (for posterity)
Comment 2 ggn 2022-07-23 12:14:05 CDT
Created attachment 188 [details]
My patch (including doc updates and some bits here and there)
Comment 3 ggn 2022-07-25 07:45:00 CDT
Created attachment 192 [details]
Second try (apparently some people define "align" as a macro)
Comment 4 Shamus Hammons 2022-08-15 21:14:56 CDT
Well, the patch doesn't cleanly apply from HEAD (currently at db581e2d9efe477db3a70fb8e947f3d009545d09).  And of course, git doesn't tell you where in the patch it fails.  :-(
Comment 5 ggn 2022-08-16 01:27:59 CDT
Created attachment 198 [details]
No idea what happened to the previous one. Absolutely none
Comment 6 ggn 2022-08-16 01:28:43 CDT
Okay, I remade the patch because I've 0 ideas what happened there. This should apply cleanly...
Comment 7 Shamus Hammons 2022-08-16 12:23:22 CDT
All right, we have a winner!  :-D