[PATCH] D89219: [ValueTracking] Use assume's noundef operand bundle

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

All of this makes sense to me. Testing coverage is low though.



================
Comment at: llvm/lib/Transforms/Utils/AssumeBundleBuilder.cpp:63
       return false;
   }
 }
----------------
We should have a test for this I think. I imagine inlining a noundef attribute with knowledge retention enabled should actually cause an assume now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89219