[PATCH] D123531: [GlobalsModRef][FIX] Ensure we honor synchronizing effects of intrinsics

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

Did you mean to drop the `!F->isIntrinsic()` check?  I mean, I guess you could replace it with `!F->isDeclaration()`, but I think you need some protection against optnone function definitions, since we don't analyze their bodies.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123531