[all-commits] [llvm/llvm-project] 796512: [Attributor][FIX] Properly return changed if the I...

Johannes Doerfert via All-commits all-commits at lists.llvm.org
Mon Sep 7 21:41:03 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 79651265b2e08e105f3d66d5f75bc9f5fa803e45
      https://github.com/llvm/llvm-project/commit/79651265b2e08e105f3d66d5f75bc9f5fa803e45
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2020-09-07 (Mon, 07 Sep 2020)

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

  Log Message:
  -----------
  [Attributor][FIX] Properly return changed if the IR was modified

Deleting or replacing anything is certainly a modification. This caused
a later assertion in IPSCCP when compiling 400.perlbench with the new PM.
I'm not sure how to test this.


  Commit: ff70c25d76561d0789743fa9f718dcd520199a7c
      https://github.com/llvm/llvm-project/commit/ff70c25d76561d0789743fa9f718dcd520199a7c
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2020-09-07 (Mon, 07 Sep 2020)

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

  Log Message:
  -----------
  [Attributor][NFC] Expand `auto` types (clang-fix-it)


  Commit: 8637acac5a3f4688114290b524eb5154a0bcdbdf
      https://github.com/llvm/llvm-project/commit/8637acac5a3f4688114290b524eb5154a0bcdbdf
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2020-09-07 (Mon, 07 Sep 2020)

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

  Log Message:
  -----------
  [Attributor][NFC] Clang tidy: no else after continue


  Commit: e6208849c8d63690ac3489813eb13196df7ed8dc
      https://github.com/llvm/llvm-project/commit/e6208849c8d63690ac3489813eb13196df7ed8dc
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2020-09-07 (Mon, 07 Sep 2020)

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

  Log Message:
  -----------
  [Attributor][NFC] Change variable spelling


  Commit: 53e4ef7fc25903430436ce456909d97aaa0fd6b2
      https://github.com/llvm/llvm-project/commit/53e4ef7fc25903430436ce456909d97aaa0fd6b2
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2020-09-07 (Mon, 07 Sep 2020)

  Changed paths:
    M llvm/test/Transforms/Attributor/internalize.ll

  Log Message:
  -----------
  [Attributor][NFC] Cleanup internalize test case

One run line was different and probably introduced for the manually
added function attribute & name checks. We can do this with the script
and a check prefix used for the other run lines as well.


  Commit: 711bf7dcf9546fefe18d32a5772d48e7b5166f08
      https://github.com/llvm/llvm-project/commit/711bf7dcf9546fefe18d32a5772d48e7b5166f08
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2020-09-07 (Mon, 07 Sep 2020)

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

  Log Message:
  -----------
  [Attributor][FIX] Don't crash on internalizing linkonce_odr hidden functions

The CloneFunctionInto has implicit requirements with regards to the
linkage and visibility of the function. We now update these after we did
the CloneFunctionInto on the copy with the same linkage and visibility
as the original.


Compare: https://github.com/llvm/llvm-project/compare/247d02396524...711bf7dcf954


More information about the All-commits mailing list