Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build error #30

Open
GoogleCodeExporter opened this issue Jul 7, 2015 · 1 comment
Open

build error #30

GoogleCodeExporter opened this issue Jul 7, 2015 · 1 comment

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. clone the latest version of the code.
2. build per instructions
3. observe the error:

jni/jpeg/jidctfst.S: Assembler messages:
jni/jpeg/jidctfst.S:66: Error: missing ')'
jni/jpeg/jidctfst.S:66: Error: garbage following instruction -- `pld (r2,#0)'
jni/jpeg/jidctfst.S:259: Error: missing ')'
jni/jpeg/jidctfst.S:259: Error: garbage following instruction -- `pld (sp,#32)'
jni/jpeg/jidctfst.S:271: Error: missing ')'
jni/jpeg/jidctfst.S:271: Error: garbage following instruction -- `pld (ip,#32)'
make: *** [obj/local/armeabi-v7a/objs/jpeg/jidctfst.o] Error 1

What is the expected output? What do you see instead?

expected a completed build, got the error show above instead.


What version of the product are you using? On what operating system?

Android NDK r8.


Please provide any additional information below.

I was able to fix this problem by changing the paranthesis to brackets on all 
PLD lines :

the PLD lines should read 

PLD     [p1, p2]

and not 

PLD     (p1, p2)


Original issue reported on code.google.com by coderroa...@gmail.com on 7 Mar 2013 at 3:15

@GoogleCodeExporter
Copy link
Author

Confirmed.  Here is the diff the compiled for me:

diff --git a/p/jni/jpeg/jidctfst.S b/p/jni/jpeg/jidctfst.S
index 34e1c24..b35f938 100644
--- a/p/jni/jpeg/jidctfst.S
+++ b/p/jni/jpeg/jidctfst.S
@@ -63,7 +63,7 @@


 jpeg_idct_ifast:
-    PLD     (r2, #0)
+    PLD     [r2, #0]
     stmdb   sp!, {r4,r5, r6,r7, r8,r9, r10,r11, r12,lr}
     ldr     r4, [sp, #4*10]
     sub     sp, #local_SIZE
@@ -256,7 +256,7 @@ VLoopHead:

 HLoopStart:
     // reset pointers
-    PLD     (sp, #off_WORKSPACE)
+    PLD     [sp, #off_WORKSPACE]
     add     ip, sp, #off_WORKSPACE
     ldr     r10, local_RANGE_TABLE

@@ -268,7 +268,7 @@ HLoopTail:
     str      r0, local_OUTPUT_BUF
     add      fp, r2, r1

-    PLD      (ip, #32)
+    PLD      [ip, #32]
     ldmia    ip!, {r0-r7}

     cmp      r1, #0


Original comment by elje...@gmail.com on 7 Mar 2013 at 11:38

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant