diff options
| author | Florian Hahn <flo@fhahn.com> | 2024-04-30 12:32:29 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-04-30 12:32:29 +0100 |
| commit | f3ac55fab8c30e71987120f76da70b5b6e9075f7 (patch) | |
| tree | 6c74fdc2f1f87a33f510847076703140fd84d864 /llvm/lib/Bitcode/Reader/BitcodeReader.cpp | |
| parent | 6fab3f2a2b04048aaa7d76d067f6cd4704bb4002 (diff) | |
| download | llvm-f3ac55fab8c30e71987120f76da70b5b6e9075f7.zip llvm-f3ac55fab8c30e71987120f76da70b5b6e9075f7.tar.gz llvm-f3ac55fab8c30e71987120f76da70b5b6e9075f7.tar.bz2 | |
[LTO] Reset DiscardValueNames in optimize(). (#78705)
libLTO parses options late, so at the moment the option is ignored. To
fix that, re-set it in optimize(), as at this point the options have been
parsed. When LTOCodeGenerator's constructor executes, the options
haven't been parsed by the linker to libLTO yet.
Note that we keep the value name of `%add = add..` because when the
module is imported, DiscardValueNames is still set to false (the default
when building with assertions).
I tried to improve this in libLTO, but I am not sure if there's a
suitable callback when all options have been set.
PR: https://github.com/llvm/llvm-project/pull/78705
Diffstat (limited to 'llvm/lib/Bitcode/Reader/BitcodeReader.cpp')
0 files changed, 0 insertions, 0 deletions