[PATCH] D64675: WIP: Disable optimization in emitStoresForConstant
Vitaly Buka via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 12 17:00:47 PDT 2019
vitalybuka added a comment.
In D64675#1584016 <https://reviews.llvm.org/D64675#1584016>, @jfb wrote:
> I would keep single stores because it's pretty silly codegen to copy from a global for that.
>
> I would also verify that `-O0` isn't affected too badly by this change.
I will. And maybe I'll measure compilation time as well.
So far I tried -O3 and -Os.
Also I tried Chromium arm, arm64, x86_64. No regression in binary size without autoinit, with zero or pattern init.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64675/new/
https://reviews.llvm.org/D64675
More information about the cfe-commits
mailing list