[meta-xilinx] [PATCH] tune-microblaze: Pin toolchain version to 4.8.4

[Manjukumar Harthikote Matha]

On 11/12/2015 05:47 AM, Nathan Rossi wrote:
> On Thu, Nov 5, 2015 at 11:21 AM, Nathan Rossi <nathan at nathanrossi.com> wrote:
>> On Thu, Nov 5, 2015 at 10:02 AM, Manjukumar Harthikote Matha
>> <manjukumar.harthikote-matha at xilinx.com> wrote:
>>> We are working on the GCC fix. Since we cannot have broken microblaze
>>> recipe, this is a temporary patch till the fix is available upstream
>>
>> Ok that's good to hear, for reference this is the standalone test case
>> that can be used to reproduce the insn missing issue
>> (https://gist.github.com/nathanrossi/93282fa77ee05703f58c). If
>> you/Xilinx find the bug or have a patch please let me know, I will do
>> the same.
>>
>> For the moment the temporary fix is to pin the version from
>> local.conf. Once the issue has been resolved the patch can be applied
>> for gcc 5.2 in meta-xilinx.
>
> Hi Manju,
>
> So I think I have found the cause and the solution for this bug. It
> appears to be caused by very specific operands of the adddi3 not
> matching the instruction definition for MicroBlaze.
>
> I have written a patch to improve 'adddi3' so that it can handle
> double word immediates. I intend to send it upstream, but it would be
> great to get a second pair of eyes and some additional testing done on
> it. I have staged this change on my nrossi/next branch:
>
Hi Nathan,

Sure, I can help on testing out the fix and will let you know the feedback.
> https://github.com/nathanrossi/meta-xilinx/blob/nrossi/next/recipes-microblaze/gcc/files/microblaze.md-Improve-adddi3-and-subdi3-insn-definit.patch
>
> Once I do some complete build tests for the various MicroBlaze
> machines I will merge it to the meta-xilinx master branch.
>
> Thanks,
> Nathan
>
Thanks
Manju

>>
>> Thanks,
>> Nathan
>>
>>>