src.opts.mk: Default LLVM_ASSERTIONS off
Needs ReviewPublic
Actions

Authored by kbowling on Sat, May 17, 12:14 AM.

Tags

None

Referenced Files

	F118458146: D50388.diff
	Thu, May 29, 3:23 PM

	F118457819: D50388.diff
	Thu, May 29, 3:17 PM

	Unknown Object (File)
	Sun, May 25, 8:03 AM

	Unknown Object (File)
	Sat, May 24, 8:44 AM

	Unknown Object (File)
	Sat, May 24, 1:21 AM

	Unknown Object (File)
	Fri, May 23, 11:15 PM

	Unknown Object (File)
	Fri, May 23, 9:15 AM

	Unknown Object (File)
	Fri, May 23, 9:13 AM

Subscribers

Details

Reviewers

dim
emaste
jrtc27

Summary

This noticeably slows down the compiler execution.

It seems like it should be toggled on in the import branches for test and exp-run and default off otherwise.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

kbowling created this revision.Sat, May 17, 12:14 AM

Herald added subscribers: ziaee, imp, bdrewery. · View Herald TranscriptSat, May 17, 12:14 AM

kbowling requested review of this revision.Sat, May 17, 12:14 AM

Or, like WITNESS or MALLOC_PRODUCTION, on in HEAD and off in stable branches?

In D50388#1149945, @jrtc27 wrote:

Or, like WITNESS or MALLOC_PRODUCTION, on in HEAD and off in stable branches?

My thinking here is few of us are LLVM developers, and it is not clear this is a benefit to apply broadly. I am less certain on where MALLOC_PRODUCTION should fall, if the thought is it catches src errors than that is a favor for keeping it on in HEAD. WITNESS is directly relevant to HEAD development and the pkgbase now makes it easy to opt out.