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

with `--errorMax:100` ; link step should not be attempted if previous step failed #9933

Closed
timotheecour opened this issue Dec 11, 2018 · 1 comment

Comments

Projects
None yet
1 participant
@timotheecour
Copy link
Contributor

commented Dec 11, 2018

(yes, I know, I introduced this flag; we/I should improve its behavior though)

I don't think it makes sense to attempt link phase when semantic failed

nim c -r --errorMax:100 tests/misc/tinvalidarrayaccess.nim

/Users/timothee/git_clone/nim/Nim/tests/misc/tinvalidarrayaccess.nim(1, 1) template/generic instantiation from here
/Users/timothee/git_clone/nim/Nim/tests/misc/tinvalidarrayaccess.nim(13, 1) template/generic instantiation from here
/Users/timothee/git_clone/nim/Nim/tests/misc/tinvalidarrayaccess.nim(18, 3) Error: index out of bounds: (a:0) <= (i:2) <= (b:1)
/Users/timothee/git_clone/nim/Nim/tests/misc/tinvalidarrayaccess.nim(1, 1) template/generic instantiation from here
/Users/timothee/git_clone/nim/Nim/tests/misc/tinvalidarrayaccess.nim(13, 1) template/generic instantiation from here
/Users/timothee/git_clone/nim/Nim/tests/misc/tinvalidarrayaccess.nim(19, 3) Error: index out of bounds: (a:0) <= (i:3) <= (b:1)
Hint:  [Link]
Undefined symbols for architecture x86_64:
  "_main", referenced from:
     implicit entry/start for main executable
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Error: execution of an external program failed: 'clang   -o /tmp/nim//app  /tmp/nim/nimcache/stdlib_system.c.o /tmp/nim/nimcache/compiler_tinvalidarrayaccess.c.o /tmp/nim/nimcache/stdlib_helpers2.c.o    -ldl'

links

#9916

@timotheecour timotheecour added the Easy label Dec 11, 2018

timotheecour added a commit to timotheecour/Nim that referenced this issue Dec 21, 2018

@timotheecour

This comment has been minimized.

Copy link
Contributor Author

commented Dec 21, 2018

=> see PR #10067

@Araq Araq closed this in #10067 Dec 21, 2018

Araq added a commit that referenced this issue Dec 21, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.