compiling assembly files with clang ia about silk-arm-ios HOT 4 CLOSED

I don't think I understand what you mean. Both ARM specific assemblies and generic C files are present, because that's how the original source code is distributed. The ARM assemblies are chosen over the generic C implementation depending on preprocessor flags.

As you can see in 92b5758, I have already made changes to make the assembly files compile under Clang. The only exception is SKP_Silk_CLZ16, which I don't understand how to fix, so I used the C version.

Which assemblies are you referring to? I would love to be able to optimize the code further, its CPU usage is still, even with the lowest complexity setting, a bit high.

from silk-arm-ios.

Which clang version are you using?
Using Apple clang version 4.0 (tags/Apple/clang-421.10.53) here..
It doesn't seem to like the macros, as well as the arm instructions
being named with conditionals in the middle (the gas way). So, to
compile with it, need to change the instruction names as mentioned
before.

On 7/22/12, Per Eckerdal
[email protected]
wrote:

I don't think I understand what you mean. Both ARM specific assemblies and
generic C files are present, because that's how the original source code is
distributed. The ARM assemblies are chosen over the generic C implementation
depending on preprocessor flags.

As you can see in 92b5758, I have already
made changes to make the assembly files compile under Clang. The only
exception is SKP_Silk_CLZ16, which I don't understand how to fix, so I
used the C version.

Which assemblies are you referring to? I would love to be able to optimize
the code further, its CPU usage is still, even with the lowest complexity
setting, a bit high.

---

Reply to this email directly or view it on GitHub:
#1 (comment)

from silk-arm-ios.

I'm using Apple clang version 3.1 (tags/Apple/clang-318.0.61) (based on LLVM 3.1svn)

This is the Clang version that is bundled with Xcode 4.3.3 (I think the compiler is bundled with Xcode, and not the iOS SDK).

Perhaps something has changed with version 4.0. If you want to, you could fork, fix it so it works for you and make a pull request. Then I will see if it works for me too and merge the fixes.

Thanks

from silk-arm-ios.

I just upgraded to Xcode 4.4, and experienced the issue you described. Your suggested changes are implemented as of 367e31c.

Thanks for helping out with this, it would have taken me ages to figure this out!

from silk-arm-ios.