[PATCH] D158527: [InstCombine] Add a cl::opt to control calls to getOrEnforceKnownAlignment in LoadInst and StoreInst
Dhruv Chawla via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 23 05:35:16 PDT 2023
0xdc03 added a comment.
In D158527#4607641 <https://reviews.llvm.org/D158527#4607641>, @goldstein.w.n wrote:
> Imo this should be in a series with the InferAlignment pass rather than as a standalone patch.
>
> Edit: typo
Hmm, I had already put this in a stack before, is a series different from that?
In D158527#4609450 <https://reviews.llvm.org/D158527#4609450>, @fhahn wrote:
> I think it would be good to try to asses the impact at least on the llvm-test-suite using statistics (see https://llvm.org/docs/TestSuiteGuide.html, `TEST_SUITE_COLLECT_STATS`) to see if there's any regressions for optimizations like DSE, GVN and others.
Will look into this.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158527/new/
https://reviews.llvm.org/D158527
More information about the llvm-commits
mailing list