Commit 4f59b0a4 authored by Nico Weber's avatar Nico Weber Committed by Commit Bot

build: Enable MemorySSA DSE with pinned clang.

The feature has been on on the tot bots for a while now and it's been
on by default upstream for a while, and things seem to work.

Since we just rolled in a clang with all the fixes, let's turn this on.

Bug: 1127713
Change-Id: I56805d576bea5936d58c653da994dc738c2a79a3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2520013
Commit-Queue: Nico Weber <thakis@chromium.org>
Reviewed-by: default avatarHans Wennborg <hans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#824947}
parent 138d9519
......@@ -490,25 +490,6 @@ config("compiler") {
"-mllvm",
"-instcombine-lower-dbg-declare=0",
]
# Pinned clang has enable-dse-memoryssa enabled by default but it's broken,
# so we have to force it off.
# Trunk clang has it disabled by default but it might work, so we force it
# on so the ToT bots can check if it works now.
if (!llvm_force_head_revision) {
cflags += [
# TODO(https://crbug.com/1127713): Investigate, remove.
"-mllvm",
"-enable-dse-memoryssa=false",
]
}
if (llvm_force_head_revision) {
cflags += [
# TODO(https://crbug.com/1127713): Investigate, remove.
"-mllvm",
"-enable-dse-memoryssa=true",
]
}
}
# C11/C++11 compiler flags setup.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment