[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