[PATCH] D26563: [LV] Stop saying "use -Rpass-analysis=loop-vectorize"

[Adam Nemet via llvm-commits]

anemet created this revision.
anemet added a reviewer: hfinkel.
anemet added a subscriber: llvm-commits.
Herald added subscribers: fhahn, mzolotukhin.

This is PR28376.

Unfortunately given the current structure of optimization diagnostics we
lack the capability to tell whether the user has
passed -Rpass-analysis=loop-vectorize since this is local to the
front-end (BackendConsumer::OptimizationRemarkHandler).

So rather than printing this even if the user has already
passed -Rpass-analysis, this patch just punts and stops recommending
this option.  I don't think that getting this right is worth the
complexity.


https://reviews.llvm.org/D26563

Files:
  lib/Transforms/Vectorize/LoopVectorize.cpp
  test/Transforms/LoopVectorize/X86/no_fpmath.ll
  test/Transforms/LoopVectorize/X86/no_fpmath_with_hotness.ll
  test/Transforms/LoopVectorize/X86/vectorization-remarks-missed.ll
  test/Transforms/LoopVectorize/control-flow.ll
  test/Transforms/LoopVectorize/diag-with-hotness-info.ll
  test/Transforms/LoopVectorize/no_switch.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26563.77668.patch
Type: text/x-patch
Size: 6175 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161111/e3614e50/attachment.bin>