[llvm] e1d564a - [llvm-exegesis] Allow retries on flaky middle half test

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 7 01:39:58 PST 2024


Author: Aiden Grossman
Date: 2024-02-07T01:38:33-08:00
New Revision: e1d564ac130fb6ec14d1e237a123373d64249e55

URL: https://github.com/llvm/llvm-project/commit/e1d564ac130fb6ec14d1e237a123373d64249e55
DIFF: https://github.com/llvm/llvm-project/commit/e1d564ac130fb6ec14d1e237a123373d64249e55.diff

LOG: [llvm-exegesis] Allow retries on flaky middle half test

The middle half repetition mode test sometimes fails on the avx512
buildbots due to a negative value being produced. This needs more
investigation, but add a retry count temporarily to alleviate
false-positive errors on the buildbots.

This is being tracked in #80957.

Added: 
    

Modified: 
    llvm/test/tools/llvm-exegesis/X86/latency/middle-half.s

Removed: 
    


################################################################################
diff  --git a/llvm/test/tools/llvm-exegesis/X86/latency/middle-half.s b/llvm/test/tools/llvm-exegesis/X86/latency/middle-half.s
index 782f906528af72..d2db493aede540 100644
--- a/llvm/test/tools/llvm-exegesis/X86/latency/middle-half.s
+++ b/llvm/test/tools/llvm-exegesis/X86/latency/middle-half.s
@@ -1,5 +1,8 @@
 # REQUIRES: exegesis-can-measure-latency, x86_64-linux
 
+# TODO(boomanaiden154): This sometimes fails with a negative latency value.
+# ALLOW_RETRIES: 2
+
 # Check that we can use the middle-half repetition mode without crashing
 
 # RUN: llvm-exegesis -mtriple=x86_64-unknown-unknown -mode=latency -opcode-name=ADD64rr -repetition-mode=middle-half-duplicate | FileCheck %s


        


More information about the llvm-commits mailing list