-
-
Notifications
You must be signed in to change notification settings - Fork 230
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
Update LDC compiler class #86
Comments
I'll fix the inline flag. Do you have any other concrete examples that need work? I'll need some time until I can build and test with the latest LDC. The "ldc2" binary name should be fine, though. Just run with "dub --compiler=ldc2". I'll make it clear in the command line help that arbitrary prefixes/suffixes to the compiler name are supported. |
I tried that, but assumed --compiler==ldc2 would use some default. No I have no other examples at the moment. I could dump the command-line help output. Then again this can probably wait. The -inline switch was actually stopping 'simple' release builds with ldc from working. |
GitHub will survive it. See if you use any of the flags incorrectly... -d-version and -d-debug ? deprectation warning flags?
|
Thanks. I've added cases for the ones I could find. Leaving this open for confirmation. |
Not quite there yet. For some reason the build process breaks when invoking the compiler. The relevant part of the output is:
When I run this line manually it works. Do you print the command verbatim? |
Sorry, I lost track of this ticket. The command line is passed via a @responsefile, so there may be something hidden - although nothing really looks suspicious... The response file is generated in the "targetPath" directory and deleted when the compiler process has exited (generators/build.d line 107). You could try to comment out that line and see what is wrong in ".dmd-response-file.txt". Anyway, I'll eventually test with LDC myself but I still have to finish a couple of other things beforehand. |
…d only quote arguments if needed. LDC seems to dislike quotes in response files (errors out with: Error: unrecognized file extension d"). Now at least builds with no spaces should work fine. See also #86.
The compiler installs as ldc2 nowadays and some of the flags need to be adapted, like
-[enable/disable]-inlining
The text was updated successfully, but these errors were encountered: