https://github.com/samolisov approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/98448
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
samolisov wrote:
@aeubanks my intent is to get no `__llvm_profile_raw_version` in modules where
nothing was actually instrumented. Usually, in my cases, they are the modules
for regular-LTO part in thin-LTO where split LTO Units are enabled. This is a
good point about consistency and using th
samolisov wrote:
It seems, github doesn't allow to modify the source branch for a PR, so I
cannot convert an already opened PR
https://github.com/llvm/llvm-project/pull/93421 into a stacked one. I've pushed
exactly the same commits into the
https://github.com/llvm/llvm-project/tree/users/psam
https://github.com/samolisov created
https://github.com/llvm/llvm-project/pull/93917
Currently, only modules that contain at least a single function
definition are instrumented. When a module contains no function
definitions at all, the module is not instrumented (yet?) and there is
no reason to
Author: Pavel Samolysov
Date: 2022-09-13T12:49:04+03:00
New Revision: 02aaf8e3d6e73116648afcbb691839ecec80aa0e
URL:
https://github.com/llvm/llvm-project/commit/02aaf8e3d6e73116648afcbb691839ecec80aa0e
DIFF:
https://github.com/llvm/llvm-project/commit/02aaf8e3d6e73116648afcbb691839ecec80aa0e.dif
Author: Pavel Samolysov
Date: 2022-09-12T15:38:11+03:00
New Revision: fe67401fd8140787b0591bb0e1de39ad78f34456
URL:
https://github.com/llvm/llvm-project/commit/fe67401fd8140787b0591bb0e1de39ad78f34456
DIFF:
https://github.com/llvm/llvm-project/commit/fe67401fd8140787b0591bb0e1de39ad78f34456.dif
