Skip to content

[Flang] Add multiline error message support to pass-plugin-not-found (NFC) #73601

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 30, 2023

Conversation

madanial0
Copy link
Contributor

The error message above has multiple lines on AIX, adding {{[[:space:]].*}} to match multiple lines

@madanial0 madanial0 added the flang Flang issues not falling into any other category label Nov 28, 2023
@madanial0 madanial0 self-assigned this Nov 28, 2023
@llvmbot
Copy link
Member

llvmbot commented Nov 28, 2023

@llvm/pr-subscribers-flang-driver

Author: None (madanial0)

Changes

The error message above has multiple lines on AIX, adding {{[[:space:]].*}} to match multiple lines


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

1 Files Affected:

  • (modified) flang/test/Driver/pass-plugin-not-found.f90 (+1-1)
diff --git a/flang/test/Driver/pass-plugin-not-found.f90 b/flang/test/Driver/pass-plugin-not-found.f90
index 08dd29e5dab0683..fc1e690a0cc72bb 100644
--- a/flang/test/Driver/pass-plugin-not-found.f90
+++ b/flang/test/Driver/pass-plugin-not-found.f90
@@ -6,4 +6,4 @@
 ! RUN: not %flang_fc1 -emit-llvm -o /dev/null -fpass-plugin=X.Y %s 2>&1 | FileCheck %s --check-prefix=ERROR
 
 ! The exact wording of the error message depends on the system dlerror.
-! ERROR: error: unable to load plugin 'X.Y': 'Could not load library 'X.Y': {{.*}}: {{.*}}{{[Nn]}}o such file{{.*}}'
+! ERROR: error: unable to load plugin 'X.Y': 'Could not load library 'X.Y': {{.*}}{{[[:space:]].*}}{{.*}}: {{.*}}{{[Nn]}}o such file{{.*}}'

Copy link
Contributor

@tblah tblah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

Copy link
Contributor

@banach-space banach-space left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

Is there any public testing on AIX for LLVM Flang?

@madanial0
Copy link
Contributor Author

LGTM, thanks!

Is there any public testing on AIX for LLVM Flang?

Not at the moment the goal is to eventually have one

@madanial0 madanial0 merged commit 586986a into llvm:main Nov 30, 2023
@madanial0 madanial0 deleted the pass-plugin-error branch November 30, 2023 06:35
Guzhu-AMD pushed a commit to GPUOpen-Drivers/llvm-project that referenced this pull request Dec 7, 2023
Local branch amd-gfx b8966b1 Merged main:770dc47659d4 into amd-gfx:8ac93c272270
Remote branch main 586986a [Flang] Add multiline error message support to pass-plugin-not-found (NFC) (llvm#73601)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flang:driver flang Flang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants