[PATCH] D92638: [Flang][openmp]Fix crash in OpenMP semantic check( bug 48308)
sameeran joshi via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 4 09:44:35 PST 2020
sameeranjoshi added inline comments.
================
Comment at: flang/test/Semantics/bug48308.f90:1
+! RUN: %S/test_errors.sh %s %t %f18 -fopenmp
+
----------------
tskeith wrote:
> I don't like the name of this test. The name should indicate what functionality it is testing, like every other test.
I was thinking similar to the way GNU compiler captures them (`pr48308.f90`) so as to find the test quickly from the name on bugzilla.
Will add a better name.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92638/new/
https://reviews.llvm.org/D92638
More information about the llvm-commits
mailing list