[PATCH] D38232: [MergeICmps] Disable mergeicmps if the target does not want to handle memcmp expansion.

[Sanjay Patel via Phabricator via llvm-commits]

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

LGTM.



================
Comment at: lib/Transforms/Scalar/MergeICmps.cpp:597-599
 class MergeICmps : public FunctionPass {
  public:
   static char ID;
----------------
courbet wrote:
> spatel wrote:
> > The indentation is non-standard; you should run clang-format before this patch to cleanup.
> I don't know why, but clang-format keeps indenting "public" and "private" to one space (even after I fix these manually). Anything I missed ?
Strange - when I run clang-format locally, it puts the 'public' at the left edge (no indent). Oh well, it was just a nit. :)


https://reviews.llvm.org/D38232