[PATCH] D130791: [clang] Short-circuit trivial constexpr array constructors
David Blaikie via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 1 14:08:01 PDT 2022
dblaikie added a comment.
> Thank you for working on this! This should also include test coverage (hmmm, we don't have a reasonable way to lit test performance regressions though.... so perhaps we just need to ensure there's at least one test that would normally have been unreasonably slow?) and a release note.
I think generally for pure-perf fixes we don't include any testing. (like switching a container type, adding caching, etc) But may be worth double-checking that there's reasonable test coverage for the feature.
also: What about zero length arrays? (which are supported as an extension) - this change would cause failures that wouldn't be emitted in the old code?
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits