Page MenuHomePhabricator

Add a 'no-asserts' requirement option to LIT.

Authored by resistor on Feb 3 2015, 11:03 AM.



This is useful when checking diagnostics and annotations that are only enabled in asserts mode. Today we can only test the positive case (when asserts are enabled) but not the negative case.

Diff Detail


Event Timeline

resistor updated this revision to Diff 19251.Feb 3 2015, 11:03 AM
resistor retitled this revision from to Add a 'no-asserts' requirement option to LIT..
resistor updated this object.
resistor edited the test plan for this revision. (Show Details)
resistor added reviewers: chandlerc, mehdi_amini.
resistor set the repository for this revision to rL LLVM.
resistor added a subscriber: Unknown Object (MLST).

Do we document the possible arguments to REQUIRES somewhere?
I'm always having to troll through the lit code to figure them out.

chandlerc requested changes to this revision.Mar 29 2015, 4:45 PM
chandlerc edited edge metadata.

I don't understand the requested change...

We already have the ability to say 'REQUIRES: asserts' when checking things only enabled in asserts builds.

It sounds like you want to check for things only enabled in *no-*asserts builds? I'm surprised any such things exist. My inclination is to say "we shouldn't have no-asserts emit data that we don't emit in asserts".

This revision now requires changes to proceed.Mar 29 2015, 4:45 PM
resistor abandoned this revision.Apr 1 2015, 9:51 AM