[llvm-commits] Fwd: Re: LDRB_POST_IMM crash

[Stepan Dyatkovskiy]

Hi Silviu,
Updated in attachment.
-Stepan.

Silviu Baranga wrote:
> Hi Stepan,
>
> The trailing CRs are around the diff headers.
>
> Could you move the ldrb CHECK line just above the memcpy call
> and add a CHECK: main: line just above the main function?
>
> Cheers,
> Silviu
>
>> -----Original Message-----
>> From: Stepan Dyatkovskiy [mailto:stpworld at narod.ru]
>> Sent: 08 October 2012 17:44
>> To: Silviu Baranga
>> Cc: llvm-commits
>> Subject: Re: [llvm-commits] LDRB_POST_IMM crash
>>
>> Hi Silviu,
>>
>> Silviu Baranga wrote:
>>
>>> The test needs some CHECK lines or else it will not pass.
>>> I suggest checking that the ldrb instruction was actually
>>> generated.
>> Sure. Patch fixed and reattached.
>>
>>> Some minor issues:
>>> The patch has trailing CRs.
>> Specially checked in hex editor. I found the only 0xA symbol at EOF...
>>
>> -Stepan.
>
>
>


-------------- next part --------------
A non-text attachment was scrubbed...
Name: ldrb_post_imm.patch
Type: text/x-patch
Size: 1713 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121010/bc923337/attachment.bin>