[PATCH] D107041: [Flang] Ported test_symbols to Python

[Andrzej Warzynski via Phabricator via llvm-commits]

awarzynski added a comment.

In D107041#2952574 <https://reviews.llvm.org/D107041#2952574>, @ashe wrote:

> The new script doesn't produce anything like the output of the old one, which was a lot more readable.
>
> If I change the second occurrence of "Implicit" to "xxx" in symbol01.f90 to make it fail, the output starts like this.
>
>   Command Output (stdout):
>   --
>   ---Expectedoutput
>   +++Actualoutput
>   @@-1,586+1,591@@
>   !DEF:/mModule
>   modulem
>   !DEF:/m/fPRIVATE,PURE,RECURSIVE(Function)SubprogramREAL(4)
>   private::f
>   contains
>   !DEF:/m/sBIND(C),PUBLIC,PURE(Subroutine)Subprogram
>   !DEF:/m/s/xINTENT(IN)(Implicit)ObjectEntityREAL(4)
>   !DEF:/m/s/yINTENT(INOUT)(-x-x-x+I+m+p+l+i+c+i+t)ObjectEntityREAL(4)
>   puresubroutines(x,y)bind(c)
>
> With the old script it was
>
>   Command Output (stdout):
>   --
>    !DEF: /m Module
>    module m
>     !DEF: /m/f PRIVATE, PURE, RECURSIVE (Function) Subprogram REAL(4)
>     private :: f
>    contains
>     !DEF: /m/s BIND(C), PUBLIC, PURE (Subroutine) Subprogram
>     !DEF: /m/s/x INTENT(IN) (Implicit) ObjectEntity REAL(4)
>   - !DEF: /m/s/y INTENT(INOUT) (xxx) ObjectEntity REAL(4)
>   + !DEF: /m/s/y INTENT(INOUT) (Implicit) ObjectEntity REAL(4)
>     pure subroutine s (x, y) bind(c)

Fixed in https://reviews.llvm.org/D107954.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107041/new/

https://reviews.llvm.org/D107041