[all-commits] [llvm/llvm-project] 7b27c0: [StackSafety, NFC] Don't test terminators

Vitaly Buka via All-commits all-commits at lists.llvm.org
Fri Jun 19 02:32:41 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 7b27c09f63526ebfd974ce4e29ef3d26ef6ab686
      https://github.com/llvm/llvm-project/commit/7b27c09f63526ebfd974ce4e29ef3d26ef6ab686
  Author: Vitaly Buka <vitalybuka at google.com>
  Date:   2020-06-19 (Fri, 19 Jun 2020)

  Changed paths:
    M llvm/lib/Analysis/StackLifetime.cpp
    M llvm/test/Analysis/StackSafetyAnalysis/lifetime.ll

  Log Message:
  -----------
  [StackSafety,NFC] Don't test terminators

Code does not track terminators and do not expose them through interface.
State there is just a state of the last instruction or entry.
So this information is just redundant and doesn't need to be tested.


  Commit: 20b1094a04a01c3f3b7c9edd071be8a30945af11
      https://github.com/llvm/llvm-project/commit/20b1094a04a01c3f3b7c9edd071be8a30945af11
  Author: Vitaly Buka <vitalybuka at google.com>
  Date:   2020-06-19 (Fri, 19 Jun 2020)

  Changed paths:
    M llvm/include/llvm/Analysis/StackLifetime.h
    M llvm/lib/Analysis/StackLifetime.cpp

  Log Message:
  -----------
  [StackSafety,NFC] Replace map with vector

We don't need to lookup InstructionNumbering by number, so
we can use vector with index as assigned number.


  Commit: 306c257b00ba4a6ef81d25897c6a05f30a9b8b2b
      https://github.com/llvm/llvm-project/commit/306c257b00ba4a6ef81d25897c6a05f30a9b8b2b
  Author: Vitaly Buka <vitalybuka at google.com>
  Date:   2020-06-19 (Fri, 19 Jun 2020)

  Changed paths:
    M llvm/include/llvm/Analysis/StackLifetime.h
    M llvm/lib/Analysis/StackLifetime.cpp
    M llvm/test/Analysis/StackSafetyAnalysis/lifetime.ll

  Log Message:
  -----------
  [SafeStack,NFC] Print liveness for all instrunctions


  Commit: f224f3d0f2b8a66ee13a26b7075e1b93bcb97dc0
      https://github.com/llvm/llvm-project/commit/f224f3d0f2b8a66ee13a26b7075e1b93bcb97dc0
  Author: Vitaly Buka <vitalybuka at google.com>
  Date:   2020-06-19 (Fri, 19 Jun 2020)

  Changed paths:
    M llvm/include/llvm/Analysis/StackLifetime.h
    M llvm/lib/Analysis/StackLifetime.cpp

  Log Message:
  -----------
  [StackSafety] Add StackLifetime::isAliveAfter

This function is going to be added into StackSafety checks.
This patch uses function in ::print implementation to make sure
that it works as expected.


Compare: https://github.com/llvm/llvm-project/compare/f9d8e33c324d...f224f3d0f2b8


More information about the All-commits mailing list