[all-commits] [llvm/llvm-project] 332e6a: [flang]Semantics for SELECT RANK.

[sameeranjoshi via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 332e6aea37a2777c06eb7d0960edf33b5869b0de
      https://github.com/llvm/llvm-project/commit/332e6aea37a2777c06eb7d0960edf33b5869b0de
  Author: sameeran joshi <sameeranjayant.joshi at amd.com>
  Date:   2020-05-08 (Fri, 08 May 2020)

  Changed paths:
    M flang/include/flang/Evaluate/shape.h
    M flang/include/flang/Semantics/symbol.h
    M flang/lib/Evaluate/shape.cpp
    M flang/lib/Semantics/CMakeLists.txt
    A flang/lib/Semantics/check-select-rank.cpp
    A flang/lib/Semantics/check-select-rank.h
    M flang/lib/Semantics/resolve-names.cpp
    M flang/lib/Semantics/semantics.cpp
    M flang/lib/Semantics/symbol.cpp
    A flang/test/Semantics/select-rank.f90
    A flang/test/Semantics/select-rank02.f90

  Log Message:
  -----------
      [flang]Semantics for SELECT RANK.

    Summary:
    Initially on github I worked on semantic checks.Then I tried some compile-time
    test of the rank value, they were failing as there were no symbols
    generated for them inside SELECT RANK's scope.So I went further to
    add new symbol in each scope, also added the respective 'rank: '
    field for a symbol when we dump the symboltable. I added a field to
    keep track of the rank in AssocEntityDetails class.This caused shape
    analysis framework to become inconsistent. So shape analysis framework
    was updated to handle this new representation.

	 *   I added more tests for above changes.

	 *   On phabricator I addressed some minor changes.

	 *   Lastly I worked on review comments.

    Reviewers: klausler,sscalpone,DavidTruby,kiranchandramohan,tskeith,anchu-rajendran,kiranktp

    Reviewed By:klausler, DavidTruby, tskeith

    Subscribers:#flang-commits, #llvm-commits

    Tags: #flang, #llvm

    Differential Revision: https://reviews.llvm.org/D78623