Skip to content

Assembler No Longer Support #3059

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Coding-Badly opened this issue Apr 29, 2015 · 6 comments
Closed

Assembler No Longer Support #3059

Coding-Badly opened this issue Apr 29, 2015 · 6 comments
Assignees
Labels
Component: IDE The Arduino IDE
Milestone

Comments

@Coding-Badly
Copy link

  • Save a bare minimum sketch.
  • Add a dot-S file to the folder; the file can be empty.
  • Close then reopen the IDE.
  • Open the recently saved bare minimum sketch.
  • Verify.

The IDE makes no attempt to include the assembler file in the build. The platform.txt file includes entries for dot-S files implying they are supported.

Additional information: With the IDE still open...

  • Click the down-arrow on the far right side of the window.
  • Click New Tab.
  • Enter a filename with a dot-S suffix.
  • The IDE complains that: ".s" is not a valid extension.
@PaulStoffregen
Copy link
Contributor

Arduino has never supported .s assembler files in sketches. They are supported in libraries and the core library, but not sketches.

I have a patch that adds this support for sketches. If the Arduino devs say they'd like to add this, I'd be happy to prepare a pull request.

This has been discussed before. I believe the decision, by David Mellis, was not to support .s files in sketch. As a general rule, I don't put time into pull requests for features that have been previously rejected. I just maintain the patches and update them for each Arduino release. Of course, I'm happy to contribute if the feature is wanted.

@Coding-Badly
Copy link
Author

Thank you for the post!

Huh. I know René Bohne used some assembly in LumiNet. For some reason I got it my head the code was delivered as a sketch but that cannot possibly be true. LumiNet used the t84 processor.

In any case, there is a person on the forum who really wants assembly support. I will suggest packaging the assembly as a library.

@q2dg
Copy link

q2dg commented Apr 30, 2015

Yes. If it's possible and it's done, why not?

@PaulStoffregen
Copy link
Contributor

Maybe @damellis could explain his thinking about why Arduino doesn't support .s files in sketches?

I would agree with "why not". In fact, I put this patch into my collection of unofficial patches for Teensy, because a few different people requested it. If you want to use .s files in a sketch, you can install Teensyduino. Most of my patches check if a Teensy board is selected and don't alter Arduino's behavior, but for this one I didn't bother with the extra checking. It should allow using .s file in sketches for any type of board.

@PaulStoffregen
Copy link
Contributor

Even without the .s file support, you CAN use inline assembly. It's a different coding style that many people who want to just use assembly don't like, but really it's about the same if you merely create empty functions to serve as containers for assembly.

@ffissore
Copy link
Contributor

Fixed in #3227. Available since next hourly build http://www.arduino.cc/en/Main/Software#hourly

@ffissore ffissore added the Component: IDE The Arduino IDE label May 26, 2015
@ffissore ffissore added this to the Release 1.6.5 milestone May 26, 2015
@ffissore ffissore self-assigned this May 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: IDE The Arduino IDE
Projects
None yet
Development

No branches or pull requests

4 participants