Closed Bug 716626 Opened 13 years ago Closed 12 years ago

Add clang's analyzer_noreturn attribute to JS_Assert

Categories

(Core :: MFBT, defect)

Product:
Core
Component:
MFBT
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED INCOMPLETE

People

(Reporter: Waldo, Unassigned)

References

(Depends on 1 open bug)

Details

See http://clang.llvm.org/docs/LanguageExtensions.html and note this doesn't affect codegen, so there's no downside whatsoever to doing this, and no auditing of existing code is needed.
Bug 716917 proposes a MOZ_DEBUGGABLE_NORETURN macro attribute to encapsulate this.
Depends on: 716917
No longer needed, as MOZ_ASSERT and MOZ_CRASH now have noreturn semantics as needed.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.