[PATCH] D96663: [InstCombine] Fold nonnull (select c,null,p) to nonnull p

Juneyoung Lee via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 22 19:49:05 PST 2021


aqjune added a comment.

ATM I don't have a clear idea about how to implement it in ValueTracking without relying on assume..

BTW, before diving into ValueTracking's update, I made D97244 <https://reviews.llvm.org/D97244> to make SimplifyCFG slightly stronger. As @ojeda mentioned in the bug report, dereferenceable implies noundef, and the attribute is pretty common, so I think it is a good idea to update this first and see how it goes.
@ojeda Is it possible to compile Rust using a custom LLVM build? Then, maybe I can play with this a bit.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96663



More information about the llvm-commits mailing list