Page MenuHomeFreeBSD

Use the compiler driver instead of ${AS}
ClosedPublic

Authored by emaste on Jul 19 2022, 7:40 PM.
Tags
None
Referenced Files
F102903759: D35859.diff
Mon, Nov 18, 1:26 PM
Unknown Object (File)
Sun, Oct 20, 11:36 AM
Unknown Object (File)
Sep 30 2024, 2:37 AM
Unknown Object (File)
Sep 28 2024, 6:53 PM
Unknown Object (File)
Sep 27 2024, 10:27 PM
Unknown Object (File)
Sep 27 2024, 5:38 PM
Unknown Object (File)
Sep 26 2024, 11:55 AM
Unknown Object (File)
Sep 26 2024, 2:56 AM

Details

Summary

As of commit fd71da37d478 we no longer have an as in the default toolchain and there is no value in keeping rules that use it.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Yea, we have all .S files at this point

This revision is now accepted and ready to land.Jul 20 2022, 3:01 AM

Note that D35864 needs to be addressed before this could be committed, and I think this warrants an exp-run as well.

Maybe the rules for .s files should be updated to compile them using CC instead?

Maybe the rules for .s files should be updated to compile them using CC instead?

Yeah @jrtc27 also made that suggestion in D35864 and it makes sense; I will update. Although nothing in the base system uses the .s.o rules now there may be ports that do and it doesn't hurt to have the rules still.

share/mk/bsd.compat.mk
40

This can be removed either way, since we won't use $AS in the base system

emaste retitled this revision from Remove ${AS} make rules to Use the compiler driver instead of ${AS}.
  • replace as rules with cc -x assembler
This revision now requires review to proceed.Jul 26 2022, 4:56 PM

I think the -x flag is not required since that is the default for .s but I don't think it does any harm.

This revision is now accepted and ready to land.Jul 26 2022, 9:25 PM
This revision was automatically updated to reflect the committed changes.