[PATCH] D158527: [InstCombine] Add a cl::opt to control calls to getOrEnforceKnownAlignment in LoadInst and StoreInst
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 25 07:11:49 PDT 2023
nikic added a comment.
In D158527#4610872 <https://reviews.llvm.org/D158527#4610872>, @0xdc03 wrote:
> Okay, @fhahn I have collected the results for both cases, can you please tell me which metrics to check?
I'd just diff all the stats to start with. I use this script that: https://gist.github.com/nikic/812f9ac1a51e29ef453fed04a6d8c40f I'm not sure this will be particularly insightful for this patch.
Another approach is to look at IR diffs. I use this patch for that purpose: https://gist.github.com/nikic/da2c7ee8120c3e477f5afc662d531b66 And then commit the results for the baseline and the change and then git diff between them.
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