[all-commits] [llvm/llvm-project] 217110: [OpenMP] Allow the Attributor to look at functions...

Johannes Doerfert via All-commits all-commits at lists.llvm.org
Sun Sep 11 20:17:19 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 21711039e3ab3fd9ed296cc2c07a011c09dd170b
      https://github.com/llvm/llvm-project/commit/21711039e3ab3fd9ed296cc2c07a011c09dd170b
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2022-09-11 (Sun, 11 Sep 2022)

  Changed paths:
    M llvm/include/llvm/Transforms/IPO/Attributor.h
    M llvm/lib/Transforms/IPO/AttributorAttributes.cpp
    M llvm/lib/Transforms/IPO/OpenMPOpt.cpp
    M llvm/test/Transforms/OpenMP/remove_globalization.ll

  Log Message:
  -----------
  [OpenMP] Allow the Attributor to look at functions we also internalized

This is important as we have accesses to globals in those which we need to
categorize.


  Commit: b046ebdc01913668361009b2fc259302d2329f29
      https://github.com/llvm/llvm-project/commit/b046ebdc01913668361009b2fc259302d2329f29
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2022-09-11 (Sun, 11 Sep 2022)

  Changed paths:
    M llvm/lib/Transforms/IPO/AttributorAttributes.cpp
    A llvm/test/Transforms/Attributor/reduced/crash_ptr2int_cgscc.ll
    M llvm/test/Transforms/OpenMP/parallel_region_merging.ll

  Log Message:
  -----------
  [Attributor][FIX] Conservatively handle ptr2int, don't crash

If a pointer-2-int cast is found we give up on AAPointerInfo for now.
This caused a crash before.

Reported by John Tramm (@jtramm).


Compare: https://github.com/llvm/llvm-project/compare/a3c05df16bc6...b046ebdc0191


More information about the All-commits mailing list