Re: [PATCH] Testsuite: mark IPA test as requiring alias support

[FX Coudert <fxcoudert@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 245 bytes --]

Hi,

> IMO, changes like this qualify as obvious, so I’d go ahead (thanks for this test fail triage)

Makes sense. I’ve committed, as well as another one, attached, slightly amending the expected pattern of a sarif plugin test.

FX


[-- Attachment #2: 0001-Testsuite-plugin-make-testcase-pattern-more-flexible.patch --]
[-- Type: application/octet-stream, Size: 1525 bytes --]

From 7be59c50478462a4bba81cddb5d2af038d4c1b1a Mon Sep 17 00:00:00 2001
From: Francois-Xavier Coudert <fxcoudert@gmail.com>
Date: Sun, 20 Aug 2023 19:35:18 +0200
Subject: [PATCH] Testsuite, plugin: make testcase pattern more flexible

On Darwin, the message recorded in the sarif file contains:
  "message": {"text": "Segmentation fault: 11"}
which is different from, e.g., linux:
  "message": {"text": "Segmentation fault"}

Adjusting the testcase pattern to be a little more flexible.

gcc/testsuite/ChangeLog:
	* gcc.dg/plugin/crash-test-write-though-null-sarif.c: Update
	expected pattern.
---
 .../gcc.dg/plugin/crash-test-write-though-null-sarif.c          | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/testsuite/gcc.dg/plugin/crash-test-write-though-null-sarif.c b/gcc/testsuite/gcc.dg/plugin/crash-test-write-though-null-sarif.c
index 83b38d2ffb5..513d66c528d 100644
--- a/gcc/testsuite/gcc.dg/plugin/crash-test-write-though-null-sarif.c
+++ b/gcc/testsuite/gcc.dg/plugin/crash-test-write-though-null-sarif.c
@@ -61,4 +61,4 @@ void test_inject_write_through_null (void)
                { dg-final { scan-sarif-file "\"startColumn\": 3" } }
                { dg-final { scan-sarif-file "\"endColumn\": 31" } }
          { dg-final { scan-sarif-file "\"message\": " } }
-           { dg-final { scan-sarif-file "\"text\": \"Segmentation fault\"" } } */
+           { dg-final { scan-sarif-file "\"text\": \"Segmentation fault" } } */
-- 
2.39.2 (Apple Git-143)