From 1499c8a4902652b17c67eab37788a8e13ef2cc82 Mon Sep 17 00:00:00 2001 From: David Major Date: Thu, 11 Feb 2021 17:13:09 +0000 Subject: [PATCH] Bug 1692234 - Allow new pass manager with ubsan on clang 10.0.1+ r=firefox-build-system-reviewers,mhentges The hang in 10.0.0 was fixed in trunk 11 and backported to 10.0.1. Differential Revision: https://phabricator.services.mozilla.com/D104836 --- build/moz.configure/flags.configure | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/build/moz.configure/flags.configure b/build/moz.configure/flags.configure index 3cad466237ab..81a3652f59bc 100644 --- a/build/moz.configure/flags.configure +++ b/build/moz.configure/flags.configure @@ -57,9 +57,9 @@ def new_pass_manager_flags(enabled, compiler, host, target, pgo, enable_fuzzing, if enable_fuzzing and compiler.version < "10.0.0": # Clang 9 does not seem to play well with libFuzzer return None - if ubsan and compiler.version >= "10.0.0": - # Temporary until https://bugs.llvm.org/show_bug.cgi?id=45835 gets a - # real fix: clang 10 hangs with some ubsan-inserted code constructs. + if ubsan and compiler.version == "10.0.0": + # Clang 10.0.0 hangs with some ubsan-inserted code constructs. + # This was fixed in 10.0.1 (https://llvm.org/pr45835) return None if enabled and compiler.version >= "9.0.0": if compiler.type == "clang":