[PATCH] D128128: [GlobalOpt] Perform store->dominated load forwarding for stored once globals

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks updated this revision to Diff 438853.
aeubanks added a comment.

remove check for thread_local global. GlobalStatus::analyzeGlobal() actually already bails if it sees a store of a thread_local variable
keep thread_local tests just to make sure


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128128

Files:
  llvm/lib/Transforms/IPO/GlobalOpt.cpp
  llvm/test/Transforms/GlobalOpt/shrink-global-to-bool-check-debug.ll
  llvm/test/Transforms/GlobalOpt/shrink-global-to-bool.ll
  llvm/test/Transforms/GlobalOpt/stored-once-forward-value.ll
  llvm/test/Transforms/PhaseOrdering/recompute-globalsaa.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D128128.438853.patch
Type: text/x-patch
Size: 7697 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220621/04d6fbb4/attachment.bin>