[Tests][LoopDistribute] Re-adding target triple in a test

[nilanjana87]

This triple was removed earlier to fix build errors because of target-specific cost model. It seems more correct to add target requirement for the test than remove it.

[llvmbot]

@llvm/pr-subscribers-llvm-transforms

Author: Nilanjana Basu (nilanjana87)

Changes

This triple was removed earlier to fix build errors because of target-specific cost model. It seems more correct to add target requirement for the test than remove it.

---

Full diff: https://github.com/llvm/llvm-project/pull/82954.diff

1 Files Affected:

• (modified) llvm/test/Transforms/LoopDistribute/basic-with-memchecks.ll (+3)

diff --git a/llvm/test/Transforms/LoopDistribute/basic-with-memchecks.ll b/llvm/test/Transforms/LoopDistribute/basic-with-memchecks.ll
index 2b9f777e9f3b24..97ea2c6708dadf 100644
--- a/llvm/test/Transforms/LoopDistribute/basic-with-memchecks.ll
+++ b/llvm/test/Transforms/LoopDistribute/basic-with-memchecks.ll
@@ -1,3 +1,4 @@
+; REQUIRES: x86-registered-target
 ; RUN: opt -aa-pipeline=basic-aa -passes=loop-distribute -enable-loop-distribute -verify-loop-info -verify-dom-info -S \
 ; RUN:   < %s | FileCheck %s
 
@@ -18,6 +19,7 @@
 ;   }
 
 target datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128"
+target triple = "x86_64-apple-macosx10.10.0"
 
 @B = common global ptr null, align 8
 @A = common global ptr null, align 8
@@ -77,6 +79,7 @@ entry:
 ; CHECK: for.end:
 
 
+; VECTORIZE: mul <4 x i32>
 ; VECTORIZE: mul <4 x i32>
 ; VECTORIZE-NOT: mul <4 x i32>
 

[fhahn]

This should preserve the original test intention, LGTM, thanks!

[fhahn]

Please try to shorten the commit title to be more concise, as per https://llvm.org/docs/DeveloperPolicy.html#commit-messages

The title should be concise. Because all commits are emailed to the list with the first line as the subject, long titles are frowned upon. Short titles also look better in git log.