[PATCH] D111515: [InstCombine] Remove attributes after hoisting free above null check

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 10 14:45:34 PDT 2021


nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.

LGTM

This is not the most precise way to do it (I think it would be sufficient to drop nonnull and convert dereferenceable into dereferenceable_or_null), but I don't think there's particular value in being precise either.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111515



More information about the llvm-commits mailing list