[PATCH] D50593: ConstantMerge: merge common initial sequences

[JF Bastien via Phabricator via llvm-commits]

jfb added a comment.

Overall this is ready to review. I ran a bunch of tests and tried it out with asan, everything seems happy, but I'm a bit paranoid. I'm re-running the test-suite before / after the change to make sure everything works as I expect it to.



================
Comment at: test/Transforms/ConstantMerge/initial-match.ll:78
+
+; FIXME: test that two zeroinitializer things get common initial sequence deduplication. Ditto undef.
+
----------------
I'll look into this soon :)


Repository:
  rL LLVM

https://reviews.llvm.org/D50593