[flang-commits] [flang] [Flang][OpenACC][OpenMP] Always add extent to acc/omp.BoundsOp (PR #73790)
Valentin Clement バレンタイン クレメン via flang-commits
flang-commits at lists.llvm.org
Fri Dec 1 09:33:45 PST 2023
clementval wrote:
> Thank you for the comment and good description of a way to fix it, I've made an attempt at fixing it in the recent push, so please do have a look to see if it addresses the previous problem.
You still have an issue with the extent you replace with 0 now. It seems wierd that an assumed shape arrays has an extent defined as `fir.undef`. The extent is computable from lb and ub.
Also on a side note, you should not force push to a PR that is under review. It makes the new changes hard to spot. When the PR is reviewed and accepted you are free to rebase/force push if needed.
https://github.com/llvm/llvm-project/pull/73790
More information about the flang-commits
mailing list