[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 21:40:34 PST 2018


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


================
Comment at: test/Transforms/GuardWidening/basic_explicit_guards.ll:670
+; containing loop.
+; TODO: Support widening for this case. We do it for intrinsic guards.
+define void @f_11(i32 %a, i1 %cond_0, i1 %cond_1) {
----------------
Actually I don't understand why we think that widening here is profitable in intrinsic case. Not sure if we should do it.


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

https://reviews.llvm.org/D56075





More information about the llvm-commits mailing list