[PATCH] D63444: [ThinLTO] Optimize write-only globals out

Evgeny Leviant via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 8 02:21:00 PDT 2019


That was a problem in caching constant expressions when processing loads and stores.
I've recomitted patch with fix and test case. See r365215 (and r365188 which had alignment problem)
________________________________________
От: Reid Kleckner via Phabricator <reviews at reviews.llvm.org>
Отправлено: 4 июля 2019 г. 3:53
Кому: Evgeny Leviant; tejohnson at google.com; stevenwu at apple.com; joker.eph at gmail.com
Копия: matthew.voss at sony.com; llvm at inglorion.net; dexonsmith at apple.com; arphaman at gmail.com; dany.grumberg at gmail.com; llvm-commits at lists.llvm.org; Igor Kudrin; George Rimar; jun.l at samsung.com
Тема: [PATCH] D63444: [ThinLTO] Optimize write-only globals out

CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.  If you suspect potential phishing or spam email, report it to ReportSpam at accesssoftek.com

rnk added a comment.

Just wanted to confirm, after reverting this, I was able to build and test LLD with ThinLTO enabled again, so this did cause the issue.

Unfortunately, I have not been able to come up with a reduced test case for the issue, so I will have to leave the instructions at "run check-lld with thinlto enabled".


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63444/new/

https://reviews.llvm.org/D63444





More information about the llvm-commits mailing list