This is an archive of the discontinued LLVM Phabricator instance.

Assert that DebugLoc is valid
ClosedPublic

Authored by davide on May 2 2015, 6:29 PM.

Details

Summary

Noticed when trying to fix PR23380.
I think it's better to assert than segfaulting one line below.

Diff Detail

Repository
rL LLVM

Event Timeline

davide updated this revision to Diff 24864.May 2 2015, 6:29 PM
davide retitled this revision from to Assert that DebugLoc is valid.
davide updated this object.
davide edited the test plan for this revision. (Show Details)
davide added a reviewer: dexonsmith.
davide set the repository for this revision to rL LLVM.
davide added a subscriber: Unknown Object (MLST).
davide accepted this revision.May 4 2015, 11:03 AM
davide added a reviewer: davide.

Discussed with dexonsmith on IRC.

This revision is now accepted and ready to land.May 4 2015, 11:03 AM
This revision was automatically updated to reflect the committed changes.