This is an archive of the discontinued LLVM Phabricator instance.

Accommodate line-0 records in a test
ClosedPublic

Authored by probinson on Nov 30 2016, 2:18 PM.

Details

Summary

LLVM is about to start emitting "line 0" records more often in the DWARF line table.
One test tripped over this; made the test accommodate the new records.

Diff Detail

Repository
rL LLVM

Event Timeline

probinson updated this revision to Diff 79813.Nov 30 2016, 2:18 PM
probinson retitled this revision from to Accommodate line-0 records in a test.
probinson updated this object.
probinson added a reviewer: jingham.
probinson added a subscriber: Restricted Project.

Forgot to mention this is for PR31214.

jingham edited edge metadata.Nov 30 2016, 2:43 PM

That's great. The source stepping should never have to do this by hand, so special casing the instruction stepping was perfect.

jingham accepted this revision.Nov 30 2016, 2:43 PM
jingham edited edge metadata.

And remember to check the accept revision...

This revision is now accepted and ready to land.Nov 30 2016, 2:43 PM
This revision was automatically updated to reflect the committed changes.

Thanks for helping to decipher the test result!