Also remove the warning line for that these prefixes are unused.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
I'm not overly opposed to this having landed, but I'll point out the flip-side of removing these (currently) unsued prefixes is that if future codegen or test changes make use of them, we won't automatically get the benefit of deduplicated checks (in the case of shared prefixes being removed) or will have to go and manually add back a new check prefix (in the case of common prefixes no longer being sufficient due to test output diverging).
This patch is fine, and it's not a huge deal either way, but for the record my current view is I wouldn't be in favour of ripping out all future unused prefixes as soon as they appear.
Comment Actions
I think it depends on whether there's the expectation that they should be sharing lines or not