It's the first time I touch this ABI. Please forgive omissions.
|966 ↗||(On Diff #44253)|
Can you fill this function to handle some basic relocations, so that it can handle some basic programs (such as a hello world written in assembly)? You can create another patch which depends on this if you want to separate them. The point is that I don't want to leave this stub empty for a long period of time.
LGTM with a few nits.
|964 ↗||(On Diff #44355)|
Add a blank line before this line.
|1 ↗||(On Diff #44355)|
Rename basicppc.s -> basic-ppc.s
|4 ↗||(On Diff #44355)|
Above file has "REQUIRES: ppc" and here we have "REQUIRES: powerpc". Which is correct? Even if both are correct, please choose one of them and use it consistently.