GitLab

The rolled-to clang has a broken feature that's enabled by default.
We disable that feature by passing -mllvm -enable-dse-memoryssa=false.

In the first CL, we only passed that in cflags, which is normally
enough. However, in LTO builds the flag must be present at link time,
so the reland adds it to ldflags as well.

(On the ToT bots, we force enable-dse-memoryssa to true to check that
the problems with the feature are fixed at clang trunk. Don't do this
in thinlto builds -- we don't have any tot thinlto bots that run tests
anyway.)

Bug: 1128966,1128144,1126953
Change-Id: I214b31bd35560d705071e4583df7b019e9f96691
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2446093
Commit-Queue: Nico Weber <thakis@chromium.org>
Commit-Queue: Hans Wennborg <hans@chromium.org>
Auto-Submit: Nico Weber <thakis@chromium.org>
Reviewed-by: Hans Wennborg <hans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#813334}