[all-commits] [llvm/llvm-project] 9029bd: [Attributor] Don't crash if getAnalysisResultForFu...

Augie Fackler via All-commits all-commits at lists.llvm.org
Tue Jul 12 13:44:29 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9029bda0419fc710aecd03328b67457e59dcd80e
      https://github.com/llvm/llvm-project/commit/9029bda0419fc710aecd03328b67457e59dcd80e
  Author: Augie Fackler <augie at google.com>
  Date:   2022-07-12 (Tue, 12 Jul 2022)

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

  Log Message:
  -----------
  [Attributor] Don't crash if getAnalysisResultForFunction() returns null LoopInfo

I have no idea what's going on here. This code was moved
around/introduced in change cb26b01d57f5 and starts crashing with a NULL
dereference once I apply https://reviews.llvm.org/D123090. I assume that
I've unwittingly taught the attributor enough that it's able to do more
clever things than in the past, and it's able to trip on this case. I
make no claims about the correctness of this patch, but it passes tests
and seems to fix all the crashes I've been seeing.

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




More information about the All-commits mailing list