[PATCH] D85837: [GlobalOpt] Fix incorrect Modified status

David Stenberg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 2 06:03:06 PDT 2020


dstenb added a comment.

Thanks for the review!

I landed this with an assert:

  +    if (GS.Ordering == AtomicOrdering::NotAtomic) {
  +      assert(!GV->isConstant() && "Expected a non-constant global");
         GV->setConstant(true);
  +      Changed = true;
  +    }


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85837



More information about the llvm-commits mailing list