[PATCH] D75020: [AssumeBundles] Add API to fill a map from operand bundles of an llvm.assume.

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 2 10:20:58 PST 2020


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

Sorry for the delay.

Two nits but otherwise LGTM.

`if (Bundles.End - Bundles.Begin <compare> SOMETHIG)`

Might deserve a helper function or two.



================
Comment at: llvm/lib/Transforms/Utils/KnowledgeRetention.cpp:180
+}
+
 #ifndef NDEBUG
----------------
static or in the anonymous namespace.


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

https://reviews.llvm.org/D75020





More information about the llvm-commits mailing list