Skip to content

[LTO] Add optimization remarks options#850

Merged
quic-seaswara merged 3 commits intomainfrom
lto-options-opt-remarks
Mar 6, 2026
Merged

[LTO] Add optimization remarks options#850
quic-seaswara merged 3 commits intomainfrom
lto-options-opt-remarks

Conversation

@quic-akaryaki
Copy link
Contributor

Add the following options:

  • --opt-remarks-filename / --plugin-opt=opt-remarks-filename=
  • --opt-remarks-format / --plugin-opt=opt-remarks-format=
  • --opt-remarks-hotness-threshold / --plugin-opt=opt-remarks-hotness-threshold=
  • --opt-remarks-passes / --opt-remarks-passes=
  • --opt-remarks-with-hotness / --plugin-opt=opt-remarks-with-hotness

@quic-akaryaki quic-akaryaki marked this pull request as draft February 24, 2026 19:34
@lenary
Copy link
Member

lenary commented Feb 26, 2026

Would it also be possible to get LLVM statistics support?

Clang supports a -save-stats option which the driver turns into "-plugin-opt=stats-file=<filename>" for LTO. The filename is aiming to go into the StatsFile field of the lto::Config object.

@quic-akaryaki
Copy link
Contributor Author

Would it also be possible to get LLVM statistics support?

Clang supports a -save-stats option which the driver turns into "-plugin-opt=stats-file=<filename>" for LTO. The filename is aiming to go into the StatsFile field of the lto::Config object.

It is in the next commit.

@quic-akaryaki quic-akaryaki force-pushed the lto-options-opt-remarks branch from 92d917e to 5bdcb21 Compare March 4, 2026 17:00
Add the following options:
- --opt-remarks-filename / --plugin-opt=opt-remarks-filename=
- --opt-remarks-format / --plugin-opt=opt-remarks-format=
- --opt-remarks-hotness-threshold / --plugin-opt=opt-remarks-hotness-threshold=
- --opt-remarks-passes / --opt-remarks-passes=
- --opt-remarks-with-hotness / --plugin-opt=opt-remarks-with-hotness

Signed-off-by: Alexey Karyakin <akaryaki@qti.qualcomm.com>
@quic-akaryaki quic-akaryaki force-pushed the lto-options-opt-remarks branch from 5bdcb21 to 16c7cab Compare March 6, 2026 18:57
Signed-off-by: Alexey Karyakin <akaryaki@qti.qualcomm.com>
Signed-off-by: Alexey Karyakin <akaryaki@qti.qualcomm.com>
@quic-akaryaki quic-akaryaki force-pushed the lto-options-opt-remarks branch from edfaacb to 8735773 Compare March 6, 2026 19:41
@quic-akaryaki quic-akaryaki marked this pull request as ready for review March 6, 2026 20:41
@quic-akaryaki quic-akaryaki requested review from lenary March 6, 2026 20:41
@quic-seaswara quic-seaswara merged commit 4e2dbb0 into main Mar 6, 2026
3 checks passed
@quic-seaswara quic-seaswara deleted the lto-options-opt-remarks branch March 6, 2026 21:05
@lenary
Copy link
Member

lenary commented Mar 6, 2026

Thanks for adding stats file support! Very helpful :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants