[all-commits] [llvm/llvm-project] cb813a: [Attributor][NFC] Improve time trace output

[Johannes Doerfert via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: cb813ab66a174396e90f92b65d74b4d813dd9c8d
      https://github.com/llvm/llvm-project/commit/cb813ab66a174396e90f92b65d74b4d813dd9c8d
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2020-10-27 (Tue, 27 Oct 2020)

  Changed paths:
    M llvm/lib/Transforms/IPO/Attributor.cpp

  Log Message:
  -----------
  [Attributor][NFC] Improve time trace output


  Commit: 30e5a1f0be027b981e415e00691883bc52957976
      https://github.com/llvm/llvm-project/commit/30e5a1f0be027b981e415e00691883bc52957976
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2020-10-27 (Tue, 27 Oct 2020)

  Changed paths:
    M llvm/lib/Transforms/IPO/AttributorAttributes.cpp
    M llvm/test/Transforms/Attributor/memory_locations.ll
    M llvm/test/Transforms/Attributor/noalias.ll

  Log Message:
  -----------
  [Attributor][FIX] Properly check uses in the call not uses of the call

In the AANoAlias logic we determine if a pointer may have been captured
before a call. We need to look at other uses in the call not uses of the
call.

The new code is not perfect as it does not allow trivial cases where the
call has multiple arguments but it is at least not unsound and a TODO
was added.


  Commit: d504f7b91a937ba10e0b549ab8f1f5bb6c5c9a07
      https://github.com/llvm/llvm-project/commit/d504f7b91a937ba10e0b549ab8f1f5bb6c5c9a07
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2020-10-27 (Tue, 27 Oct 2020)

  Changed paths:
    M llvm/lib/Transforms/IPO/AttributorAttributes.cpp

  Log Message:
  -----------
  [Attributor][NFC] Hoist call out of a lambda

The call is not free, unsure if  this is needed but it does not make it
worse either.


  Commit: 724fcce1092da409fe4ea63fb27d23637d472680
      https://github.com/llvm/llvm-project/commit/724fcce1092da409fe4ea63fb27d23637d472680
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2020-10-27 (Tue, 27 Oct 2020)

  Changed paths:
    M llvm/lib/Transforms/IPO/Attributor.cpp

  Log Message:
  -----------
  [Attributor][NFC] Clang-format


  Commit: bfe05b1aff1b8e5dd625f905879d7425c8460989
      https://github.com/llvm/llvm-project/commit/bfe05b1aff1b8e5dd625f905879d7425c8460989
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2020-10-27 (Tue, 27 Oct 2020)

  Changed paths:
    M llvm/lib/Transforms/IPO/AttributorAttributes.cpp
    M llvm/test/Transforms/Attributor/range.ll

  Log Message:
  -----------
  [Attributor][FIX] Do not attach range metadata to the wrong Instruction

If we are looking at a call site argument it might be a load or call
which is in a different context than the call site argument. We cannot
simply use the call site argument range for the call or load.

Bug reported and reduced by Whitney Tsang <whitneyt at ca.ibm.com>.


  Commit: 1c2531c9e16412ba2bb2f8e594d7effa48e9d22e
      https://github.com/llvm/llvm-project/commit/1c2531c9e16412ba2bb2f8e594d7effa48e9d22e
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2020-10-27 (Tue, 27 Oct 2020)

  Changed paths:
    M llvm/include/llvm/Transforms/IPO/Attributor.h
    M llvm/lib/Transforms/IPO/Attributor.cpp
    M llvm/test/Transforms/Attributor/ArgumentPromotion/2008-09-08-CGUpdateSelfEdge.ll
    M llvm/test/Transforms/Attributor/IPConstantProp/deadarg.ll
    M llvm/test/Transforms/Attributor/liveness.ll

  Log Message:
  -----------
  [Attributor][FIX] Delete all unreachable static functions

Before we used to only mark unreachable static functions as dead if all
uses were known dead. Now we optimistically assume uses to be dead until
proven otherwise.


  Commit: 1d57b7f5039a8b30fa6f1a448d1417c8a7783470
      https://github.com/llvm/llvm-project/commit/1d57b7f5039a8b30fa6f1a448d1417c8a7783470
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2020-10-27 (Tue, 27 Oct 2020)

  Changed paths:
    M llvm/lib/Transforms/IPO/Attributor.cpp

  Log Message:
  -----------
  [Attributor][NFC] Print the right value in debug output


  Commit: 50d34958df7b7a335ce4ddc0258ed28273430399
      https://github.com/llvm/llvm-project/commit/50d34958df7b7a335ce4ddc0258ed28273430399
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2020-10-27 (Tue, 27 Oct 2020)

  Changed paths:
    M llvm/lib/Transforms/IPO/Attributor.cpp

  Log Message:
  -----------
  [Attributor][NFC] Introduce a debug counter for `AA::manifest`

This will simplify debugging and tracking down problems.


  Commit: d13daa4018faf6a3427d80ed17e10f5913fee1b5
      https://github.com/llvm/llvm-project/commit/d13daa4018faf6a3427d80ed17e10f5913fee1b5
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2020-10-27 (Tue, 27 Oct 2020)

  Changed paths:
    M llvm/lib/Transforms/IPO/Attributor.cpp

  Log Message:
  -----------
  [Attributor] Finalize the CGUpdater after each SCC

This matches the new PM model.


Compare: https://github.com/llvm/llvm-project/compare/035e12e66449...d13daa4018fa