https://github.com/jeanPerier closed
https://github.com/llvm/llvm-project/pull/83285
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jeanPerier updated
https://github.com/llvm/llvm-project/pull/83285
>From 8c5bd904e624c9ec10d75ddf09b8ec55c32d8b06 Mon Sep 17 00:00:00 2001
From: Jean Perier
Date: Wed, 28 Feb 2024 06:17:11 -0800
Subject: [PATCH] [flang] Enable polymorphic lowering by default
Polymorphic enti
https://github.com/jeanPerier updated
https://github.com/llvm/llvm-project/pull/83285
>From 6944f908d5ce82871323816bc247fcb7e765531f Mon Sep 17 00:00:00 2001
From: Jean Perier
Date: Wed, 28 Feb 2024 06:17:11 -0800
Subject: [PATCH] [flang] Enable polymorphic lowering by default
Polymorphic enti
tarunprabhu wrote:
> A few of the tests in the gfortran-testsuite using polymorphism fail at
> runtime. I'm happy for this PR to be merged as it is and these bugs to be
> fixed later.
Once this is merged in, the polymorphism tests can be enabled together with the
larger group of tests that I
jeanPerier wrote:
Thanks a lot for testing Tom!
I did not expect issues with gfortran tests. I will look into it before merging
and I made a simpler first patch thet will be less noisy to revert in case of
other issues that would break the build bots:
https://github.com/llvm/llvm-project/pull
tblah wrote:
A few of the tests in the gfortran-testsuite using polymorphism fail at
runtime. I'm happy for this PR to be merged as it is and these bugs to be fixed
later.
https://github.com/llvm/llvm-project/pull/83285
___
cfe-commits mailing list
c
https://github.com/tblah edited https://github.com/llvm/llvm-project/pull/83285
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tblah approved this pull request.
LGTM. On a quick run this seems to pass everything I would expect it to in the
gfortran test suite. When you get time please could you enable the polymorphic
tests in the gfortran test suite and re-categorize the ones which now hit
different
https://github.com/kiranchandramohan approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/83285
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/clementval approved this pull request.
Looks great! Thanks Jean to work on this!
https://github.com/llvm/llvm-project/pull/83285
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
@llvm/pr-subscribers-flang-fir-hlfir
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-flang-driver
Author: None (jeanPerier)
Changes
Polymorphic entity lowering status is good. The main remaining TODO is to allow
lowering of vector subscript
https://github.com/jeanPerier created
https://github.com/llvm/llvm-project/pull/83285
Polymorphic entity lowering status is good. The main remaining TODO is to allow
lowering of vector subscripted polymorphic entity, but this does not deserve
blocking all application using polymorphism.
Remov
12 matches
Mail list logo