[PATCH] D69947: [ThinLTO] Simplify attribute propagation

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 11 11:08:28 PST 2019


tejohnson added a comment.

Given the write only bug that popped up with the earlier patch, I'm wondering if this is safe. I.e. we must import a global variable if we have marked it as write only, so if we don't import it later (because it fails one of the legality checks in canImportGlobalVar), then we can end up in trouble.


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

https://reviews.llvm.org/D69947





More information about the llvm-commits mailing list