>>  I have tried to copy-and-paste the patch text into a patch file I can
>>  apply but I'm having no luck, this is the error message I'm getting:
>> 
>>   $ patch -p0 <flags.patch
>>   patching file rtl/verilog/or1200/or1200_alu.v
>>   patch: **** malformed patch at line 6: `endif
>> 
> Try the attached file which was created by piping the output of 'svn
> diff' on the file in the or1200's RTL directory.

That worked, thanks. I think it's best to send patch files as attachments. It's 
not the first time I hear in this mailing list that someone is having problems 
when the e-mail client reformats the lines.

I applied the patch and now it's failing another test:

        // Add two quite large positive numbers with a carry in. Should set the 
overflow but not the carry flag.
        TEST_INST_FF_I32_I32 SPR_SR_CY, SPR_SR_OV, l.addc, 0x40000000, 
0x3fffffff

According to the test comment, the overflow flag should be set in that case, 
but it's not after applying the patch. The test runs fine against or1ksim.

I just pushed a change to orbuild to make it easier to re-run all tests quickly 
on your local PC, just look for the following line in the sources:

# During development, it's quite convenient to rerun the tests without 
rebuilding

Regards,
  rdiez
_______________________________________________
OpenRISC mailing list
[email protected]
http://lists.openrisc.net/listinfo/openrisc

Reply via email to