Emit notice when compiling with -release

[0xEAB]

Partially revamps ldc-developers/ldc#4739 — but for DMD.

Warning: Compiling with `-release` can have undesirable side effects and is discouraged.
	Please consult the documentation for details.
	For fast binaries through strong optimization, use LDC, the LLVM-based D Compiler, instead.

Using -ludicrous instead of -release gets one rid of the notice message.

[adamdruppe]

sooooooo druntime uses it in the build. i guess kidna harmless but maybe it should be updated

[0xEAB]

I know it’s tiring, but please review these substitutions. I’ve staged them patch by patch but it is easy to miss a case where -release shouldn’t have been substituted.

[0xEAB]

3ddd480 is untested.

[adamdruppe]

actually i might have made a mistake in requesting that cuz it bootstrap builds w/ upstream....... ugh

[adamdruppe]

so yeah sorry for making you do something tedious and useless but it should prolly just keep using -release for compatibility with upstream host compilers doing the build, the message is ok to see anyway tbh

[0xEAB]

do something tedious

No big deal, running find -exec sed {} ain’t that much work.
Just the patch staging is a tad monotonous (hence the increased chance of making a mistake).

[0xEAB]

cuz it bootstrap builds w/ upstream....... ugh

But doesn’t it use the fresh DMD to build druntime + Phobos?
DMD=/upstream-dmd/generated/linux/release/64/dmd make dmd druntime phobos works for me.

[0xEAB]

oh, ENABLE_RELEASE=1 make dmd is broken, but that’s due to my patch for build.d being insufficient nonsense.

[0xEAB]

I think this should work fine now.

[adamdruppe]

well let's try it and see lol

[adamdruppe]

yea it failed so ima undo the changes to the build scripts

[adamdruppe]

actually it isn't the build script, it is the ldc commit saying redefinition. megannoying template magic

[adamdruppe]

or actually maybe just a reused variable lemme try changing that

[0xEAB]

awesome, thanks ^^