This is an archive of the discontinued LLVM Phabricator instance.

Add a virtual method on Action to indicate if they can be skipped safely
AcceptedPublic

Authored by mehdi_amini on Feb 25 2023, 10:18 PM.

Details

Summary

This can be useful to drive something like debug counters for example.

Diff Detail

Event Timeline

mehdi_amini created this revision.Feb 25 2023, 10:18 PM
mehdi_amini requested review of this revision.Feb 25 2023, 10:18 PM

This patch feels kind of weird here, why not delay this until when it can actually be used?

After looking at the followup, I'd honestly just fold this into the followup.

rriddle accepted this revision.Apr 23 2023, 11:28 PM
This revision is now accepted and ready to land.Apr 23 2023, 11:28 PM
jpienaar accepted this revision.Apr 28 2023, 4:41 AM
Matt added a subscriber: Matt.May 1 2023, 2:03 PM