[all-commits] [llvm/llvm-project] cc0526: [GlobalOpt] Fix unreachable ifunc globalopt crash ...

[Justin Riddell via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: cc052667b4b2f3bed12db68da876626e14f231f8
      https://github.com/llvm/llvm-project/commit/cc052667b4b2f3bed12db68da876626e14f231f8
  Author: Justin Riddell <arghnews at hotmail.co.uk>
  Date:   2025-09-10 (Wed, 10 Sep 2025)

  Changed paths:
    M llvm/lib/Transforms/IPO/GlobalOpt.cpp
    A llvm/test/Transforms/GlobalOpt/resolve-indirect-ifunc.ll

  Log Message:
  -----------
  [GlobalOpt] Fix unreachable ifunc globalopt crash (#157332) (#157593)

Also fixes (#131488)

Unreachable case is triggering `Callees.empty()` assert. Since this was
[originally
](https://github.com/llvm/llvm-project/pull/87939/commits/02bd5a7013c558f1e5220fc89bafa68f40276549#diff-06aba0dac2a263dc14297a15655291d5506b760f54a736385bcf3208f83df843R2524)
a `continue` anyway, have applied that as a fix and added a test case.
Please let me know if there's a better way.

Not sure who/how to get folks to review, tagging a few people (apologies
if you're not the right person/this is the wrong way to do it, please
let me know what to do in future if so)

@labrinea @dtcxzyw @nikic @fhahn



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications