[PATCH] D89854: [GVN LoadPRE] Add an option to disable splitting backedge

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

This seems reasonable. Do you plan to use this option in the default pipeline/do you think there are any benefits for the regular pipeline?



================
Comment at: llvm/test/Transforms/GVN/PRE/load-pre-split-backedge.ll:6
+target datalayout = "e-p:32:32:32-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:32:64-f32:32:32-f64:32:64-v64:64:64-v128:128:128-a0:0:64-f80:128:128-n8:16:32"
+target triple = "i386-apple-darwin11.0.0"
+
----------------
nit: do we need the triple?


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

https://reviews.llvm.org/D89854