llvm-project.git/clang/test/CodeGen/thinlto_embed_bitcode.ll, branch main Unnamed repository; edit this file 'description' to name the repository. [NFC][tests] Replace non-portable grep with FileCheck 2020-11-24T17:15:07+00:00 Hubert Tong hubert.reinterpretcast@gmail.com 2020-11-24T17:14:03+00:00 44174b3d518ed70482ff5df2879523a4e26f92cc After commit 2482648a795afbe12774168bbbf70dc14c031267, a GNU grep option is just passed unconditionally to `grep` in general. This patch fixes the test for platforms where `grep` is not GNU grep. 
After commit 2482648a795afbe12774168bbbf70dc14c031267, a GNU grep option
is just passed unconditionally to `grep` in general. This patch fixes
the test for platforms where `grep` is not GNU grep.
thinlto_embed_bitcode.ll: clarify grep should treat input as text 2020-11-22T05:46:53+00:00 Mircea Trofin mtrofin@google.com 2020-11-22T05:46:53+00:00 2482648a795afbe12774168bbbf70dc14c031267 The input to the test's use of grep should be treated as text, and that's not the case on certain Linux distros. Added --text. 
The input to the test's use of grep should be treated as text, and
that's not the case on certain Linux distros. Added --text.
[ThinLTO] Fix empty .llvmcmd sections 2020-10-29T16:57:42+00:00 Mircea Trofin mtrofin@google.com 2020-10-29T03:16:51+00:00 13aee94bc710bfa6277c1f07146c714ee65bf2de When passing -lto-embed-bitcode=post-merge-pre-opt, we were getting empty .llvmcmd sections. It turns out that is because the CodeGenOptions::CmdArgs field was only populated when clang saw -fembed-bitcode={all|marker}. This patch always populates the CodeGenOptions::CmdArgs. The overhead of carrying through in memory in all cases is likely negligible in the grand schema of things, and it keeps the using code simple. Differential Revision: https://reviews.llvm.org/D90366 
When passing -lto-embed-bitcode=post-merge-pre-opt, we were getting
empty .llvmcmd sections. It turns out that is because the
CodeGenOptions::CmdArgs field was only populated when clang saw
-fembed-bitcode={all|marker}.

This patch always populates the CodeGenOptions::CmdArgs. The overhead
of carrying through in memory in all cases is likely negligible in
the grand schema of things, and it keeps the using code simple.

Differential Revision: https://reviews.llvm.org/D90366
[ThinLTO] Fix .llvmcmd emission 2020-10-29T00:45:30+00:00 Mircea Trofin mtrofin@google.com 2020-10-28T00:22:30+00:00 735ab4be35695df9f9da7ae8b584cec28eabf1fe llvm::EmbedBitcodeInModule needs (what used to be called) EmbedMarker set, in order to emit .llvmcmd. EmbedMarker is really about embedding the command line, so renamed the parameter accordingly, too. This was not caught at test because the check-prefix was incorrect, but FileCheck does not report that when multiple prefixes are provided. A separate patch will address that. Differential Revision: https://reviews.llvm.org/D90278 
llvm::EmbedBitcodeInModule needs (what used to be called) EmbedMarker
set, in order to emit .llvmcmd. EmbedMarker is really about embedding the
command line, so renamed the parameter accordingly, too.

This was not caught at test because the check-prefix was incorrect, but
FileCheck does not report that when multiple prefixes are provided. A
separate patch will address that.

Differential Revision: https://reviews.llvm.org/D90278
[ThinLTO] Option to bypass function importing. 2020-09-22T20:12:11+00:00 Mircea Trofin mtrofin@google.com 2020-09-16T19:08:15+00:00 cf112382ddd0d717edf0b8a3e5b061de925258e9 This completes the circle, complementing -lto-embed-bitcode (specifically, post-merge-pre-opt). Using -thinlto-assume-merged skips function importing. The index file is still needed for the other data it contains. Differential Revision: https://reviews.llvm.org/D87949 
This completes the circle, complementing -lto-embed-bitcode
(specifically, post-merge-pre-opt). Using -thinlto-assume-merged skips
function importing. The index file is still needed for the other data it
contains.

Differential Revision: https://reviews.llvm.org/D87949
[ThinLTO] Relax thinlto_embed_bitcode.ll check 2020-09-16T05:42:22+00:00 Mircea Trofin mtrofin@google.com 2020-09-16T05:40:13+00:00 8ea7ef8eda93aa144c339275fc6d9db2615a0118 Fixes fuscia test [1] - the thinlto annotations may not always be there. [1] http://lab.llvm.org:8011/builders/fuchsia-x86_64-linux/builds/11312 
Fixes fuscia test [1] - the thinlto annotations may not always be there.

[1] http://lab.llvm.org:8011/builders/fuchsia-x86_64-linux/builds/11312
[ThinLTO] add post-thinlto-merge option to -lto-embed-bitcode 2020-09-15T22:56:11+00:00 Mircea Trofin mtrofin@google.com 2020-09-14T17:45:00+00:00 61fc10d6a520f267e11009ce8fce88d73615796b This will embed bitcode after (Thin)LTO merge, but before optimizations. In the case the thinlto backend is called from clang, the .llvmcmd section is also produced. Doing so in the case where the caller is the linker doesn't yet have a motivation, and would require plumbing through command line args. Differential Revision: https://reviews.llvm.org/D87636 
This will embed bitcode after (Thin)LTO merge, but before optimizations.
In the case the thinlto backend is called from clang, the .llvmcmd
section is also produced. Doing so in the case where the caller is the
linker doesn't yet have a motivation, and would require plumbing through
command line args.

Differential Revision: https://reviews.llvm.org/D87636