[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