[PATCH] D53356: [InstCombine] Teach the move free before null test opti how to deal with noop casts

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 29 09:43:20 PDT 2018


qcolombet added a comment.

Hi Sanjay,

I had a quick look and it doesn't seem to be a good fit for simplifyCFG.
In particular, simplifyCFG only inspects terminator instructions and it feels out of place to add a scan through the entire list of instructions to find a call to free.

Let me know if you want me to pursue in that direction or if you have another idea.

Cheers,
-Quentin


Repository:
  rL LLVM

https://reviews.llvm.org/D53356





More information about the llvm-commits mailing list