[flang-commits] [flang] [flang] `do concurrent`: Enable delayed localization by default (PR #154303)

[Kareem Ergawy via flang-commits]

ergawy wrote:

> Hi @ergawy, it looks like the `DoConcurrentConversion` in `SimplifyFIROperations.cpp` is failing on the following test, and your change exposed the problem:
> 
> ```
> subroutine test(a,x)
>   character(*) :: a(:),x
>   do concurrent (i=1:10) local(x)
>      x='go'
>      a(i)=x
>   end do
> end subroutine test
> ```
> 
> `flang test.f90`:
> 
> ```
> error: 'fir.unboxchar' op operand #0 must be CHARACTER type descriptor., but got '!fir.ref<!fir.boxchar<1>>'
> ```
> 
> Could you please fix it or revert the change?

Sorry for the late reply. I am looking into it, if the fix is not quick, I will rever the PR.

https://github.com/llvm/llvm-project/pull/154303