|
From: Stepan D. <stp...@na...> - 2013-07-28 18:36:19
|
Exactly. You got assertion due to incorrect inlineasm instruction. I had also failed to reproduce this behavoiur for reduced case. This bug fired for big source only. Try compile that big source with -emit-llvm, and lookup inline asm (there is some variable with name, that ends with "m10", perhaps you recheck that name in .c file). I can attach result IR file with string number reference a bit later, currently I'm stucking in traffic jam :-(
-Stepan
> I've compiled the file you attached, ignoring all warnings i get from clang about the unknown attributes like progmem, i'm getting the assertion "multibyte index is out of range."
>
> 2013/7/28 Borja Ferrer <bor...@gm...>
>
>> Hello Stepan,
>>
>> 1) It seems clang mangles assembly functions names by prefixing a \01 character. I haven't found yet the code that does this depending on the target used.
>>
>> 2) About the code above, I'm assuming you got that inline asm code from the __LPM_dword_classic__ macro. This is what I'm getting for the following C code:
>> typedef unsigned int uint16_t;
>> typedef unsigned long uint32_t;
>>
>> #define __LPM_dword_classic__(addr) \
>> (__extension__({ \
>> uint16_t __addr16 = (uint16_t)(addr); \
>> uint32_t __result; \
>> __asm__ __volatile__ \
>>
>> ( \
>> "lpm" "\n\t" \
>> "mov %A0, r0" "\n\t" \
>> "adiw r30, 1" "\n\t" \
>>
>> "lpm" "\n\t" \
>> "mov %B0, r0" "\n\t" \
>> "adiw r30, 1" "\n\t" \
>> "lpm" "\n\t" \
>>
>> "mov %C0, r0" "\n\t" \
>> "adiw r30, 1" "\n\t" \
>> "lpm" "\n\t" \
>> "mov %D0, r0" "\n\t" \
>>
>> : "=r" (__result), "=z" (__addr16) \
>> : "1" (__addr16) \
>> : "r0" \
>>
>> ); \
>>
>> __result; \
>> }))
>> unsigned long inlineasm(unsigned int addr)
>> {
>> return __LPM_dword_classic__(addr);
>> }
>>
>> clang produces:
>>
>> define i32 @inlineasm(i16 %addr) #0 {
>>
>> entry:
>> %0 = tail call { i32, i16 } asm sideeffect "lpm\0A\09mov ${0:A}, r0\0A\09adiw r30, 1\0A\09lpm\0A\09mov ${0:B}, r0\0A\09adiw r30, 1\0A\09lpm\0A\09mov ${0:C}, r0\0A\09adiw r30, 1\0A\09lpm\0A\09mov ${0:D}, r0\0A\09", "=r,=z,1,~{r0}"(i16 %addr) #2, !srcloc !4
>>
>> %asmresult = extractvalue { i32, i16 } %0, 0
>> ret i32 %asmresult
>> }
>>
>> No truncations here as far i can tell. Try this code and see what you get.
--
Truly yours,
Stepan Dyatkovskiy
|