[PATCH] D74792: [SimplifyLibCalls][IRBuilder] Accept any IRBuilder in SimplifyLibCalls

[Mikael Holmén via Phabricator via llvm-commits]

uabelho added a comment.

Hi,

Is this supposed to affect the output from instcombine?

When I merged this to our downstream clone I noticed that it does.
If I run instcombine on the following input:

  declare i32 @strlen(i8*)
  
  define void @test_it() {
  bb1:
    %_tmp12 = call i32 @strlen(i8* undef)
    ret void
  }

with

  opt -instcombine -S -o - strlen.ll

I got

  define void @test_it() {
  bb1:
    ret void
  }

before, but with this patch I now get

  define void @test_it() {
  bb1:
    store i8 undef, i8* null, align 536870912
    ret void
  }

instead.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74792/new/

https://reviews.llvm.org/D74792