[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