[PATCH v2 0/5] Add AArch64 support to libdl

Chris Johns chrisj at rtems.org
Thu Jul 28 00:00:05 UTC 2022


On 28/7/2022 6:02 am, Ryan Long wrote:
> On 7/25/2022 10:48 PM, Chris Johns wrote:
>> On 26/7/2022 12:06 am, Ryan Long wrote:
>>> In this version of the patchset I converted the tabs to spaces in
>>> rtl-mdreloc-aarch64.c.
>> I think setting the loader to NULL is a bug.
> 
> I removed it from rtl-mdreloc-arm.c, and ran dl05. It still passed. So should I
> just take this line out, and keep it out when I refactor the code?

I will patch and push the fix to ARM so it is not forgotten.

>>> I didn't address some of the other issues in rtl-mdreloc-aarch64.c due to the
>>> code being copied from rtl-mdreloc-arm.c. I can pull out the shared code from
>>> the two files in a follow up patch.
>> I see you close the #4682. What about keeping it open to track this?
> I created a new ticket that'll be for refactoring the code.

OK

Chris


More information about the devel mailing list