From 79ceda2e394cefd6ce650dc40fce44295e76be2b Mon Sep 17 00:00:00 2001 From: Erich Keane Date: Tue, 23 Jun 2020 08:33:44 -0700 Subject: [PATCH] Fix test added by D79830 This clang test unfortunately depends on the actions of the optimizer, which some of the buildbots hit. This patch makes it so it cannot ignore the return value of 'f', so it won't do away with the implementation. --- clang/test/CodeGen/builtin-expect-with-probability.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/clang/test/CodeGen/builtin-expect-with-probability.cpp b/clang/test/CodeGen/builtin-expect-with-probability.cpp index 2d25219..e63f35b 100644 --- a/clang/test/CodeGen/builtin-expect-with-probability.cpp +++ b/clang/test/CodeGen/builtin-expect-with-probability.cpp @@ -1,4 +1,5 @@ // RUN: %clang_cc1 -emit-llvm -o - %s -O1 | FileCheck %s +// RUN: %clang_cc1 -emit-llvm -o - -fexperimental-new-pass-manager %s -O1 | FileCheck %s extern int global; struct S { @@ -15,8 +16,8 @@ int expect_taken(int x) { return x; } -void f() { - expect_taken(global); +int f() { + return expect_taken(global); } int expect_taken2(int x) { -- 2.7.4