[PATCH] D124241: [Local] Consider atomic loads from constant global as dead

[Florian Hahn via Phabricator via llvm-commits]

fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.

LGTM, thanks!



================
Comment at: llvm/lib/Transforms/Utils/Local.cpp:503
 
+  // Atomic loads from constants can be removed.
+  if (auto *LI = dyn_cast<LoadInst>(I))
----------------
nit: can be removed, unless they are volatile.?


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

https://reviews.llvm.org/D124241