This is an archive of the discontinued LLVM Phabricator instance.

[Bazel] Make gentbl_test compatible with coverage
ClosedPublic

Authored by GMNGeoffrey on Jul 15 2021, 6:08 PM.

Details

Summary

Add explicit coverage provider. Also remove output_to_genfiles which
isn't necessary for this test (it's just copy-pasta from gentbl_rule,
which needs it for output C++ header files).

Diff Detail

Event Timeline

GMNGeoffrey created this revision.Jul 15 2021, 6:08 PM
GMNGeoffrey requested review of this revision.Jul 15 2021, 6:08 PM
jpienaar accepted this revision.Jul 16 2021, 1:53 PM

I thought we did this one a while ago 🙂

This revision is now accepted and ready to land.Jul 16 2021, 1:53 PM

I thought we did this one a while ago 🙂

Someone did internally. I'm backporting missing updates from the internal copy

This revision was landed with ongoing or failed builds.Jul 16 2021, 1:57 PM
This revision was automatically updated to reflect the committed changes.