[PATCH] D56075: [GuardWidening] Support widening of explicitly expressed guards

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 25 22:03:33 PST 2018


mkazantsev marked an inline comment as done.
mkazantsev added inline comments.


================
Comment at: test/Transforms/GuardWidening/basic_explicit_guards.ll:886
+
+; TODO: Support widening here. We do it for intrinsic guards.
+define void @f_14(i32 %a) {
----------------
I've re-read the test carefully. We don't do it in intrinsic case and shouldn't. TODO is to be removed.


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

https://reviews.llvm.org/D56075





More information about the llvm-commits mailing list