[PATCH] D94783: [LTO] Remove options to disable inlining and GVNLoadPRE.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 15 09:22:40 PST 2021


fhahn updated this revision to Diff 316996.
fhahn added a comment.

In D94783#2501324 <https://reviews.llvm.org/D94783#2501324>, @tejohnson wrote:

> Agree, these should just be removed. but I also don't see any uses of disable-lto-vectorization in a test, can that and the corresponding DisableVectorization parameter be nuked as well?

Sounds good, let's try to get rid of those custom options. To disable vectorization, we already use a way to works better with LTO, by adding metadata to disable vectorization.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94783

Files:
  llvm/include/llvm/LTO/legacy/LTOCodeGenerator.h
  llvm/lib/LTO/LTOCodeGenerator.cpp
  llvm/tools/llvm-lto/llvm-lto.cpp
  llvm/tools/lto/lto.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D94783.316996.patch
Type: text/x-patch
Size: 7810 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210115/7643c773/attachment.bin>


More information about the llvm-commits mailing list