[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