Page MenuHomeFreeBSD

Define 'lr' as x30 on aarch64
ClosedPublic

Authored by kan on Apr 7 2017, 3:36 PM.

Details

Summary

GNU toolchain does not recognize LR as standard register alias,
but clang does. Use of #define will work on both. Place the
definition into central machine/asm.h instead of patching every
affected file, as requested by plaftorm maintainers.

Diff Detail

Repository
rS FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

kan created this revision.Apr 7 2017, 3:36 PM
This revision is now accepted and ready to land.Apr 7 2017, 3:38 PM
andrew accepted this revision.Apr 7 2017, 3:45 PM
emaste accepted this revision.Apr 7 2017, 3:57 PM

No objection from me. I am a bit puzzled by us having a mix of lr and x30 in our .S sources though.

This revision was automatically updated to reflect the committed changes.