[PATCH] D123531: [GlobalsModRef][FIX] Ensure we honor synchronizing effects of intrinsics
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 11 12:19:42 PDT 2022
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
More information about the llvm-commits
mailing list