[PATCH] D115302: GlobalsModRef should treat functions w/o nosync conservatively.

Artem Belevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 13 15:05:07 PST 2021


tra updated this revision to Diff 394068.
tra added a comment.

Undo ObjC intrinsic properties change. Fixed the tests instead.
Moved `nosync` check into `getModRefInfo`, where it makes more
sense as an additional check for potential side effects of the call.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115302

Files:
  llvm/lib/Analysis/GlobalsModRef.cpp
  llvm/test/Analysis/GlobalsModRef/dead-uses.ll
  llvm/test/Analysis/GlobalsModRef/func-memattributes.ll
  llvm/test/Analysis/GlobalsModRef/functions_without_nosync.ll
  llvm/test/Analysis/GlobalsModRef/intrinsic_addressnottaken1.ll
  llvm/test/Analysis/GlobalsModRef/intrinsic_addressnottaken2.ll
  llvm/test/Analysis/GlobalsModRef/intrinsic_addresstaken.ll
  llvm/test/Analysis/GlobalsModRef/no-escape.ll
  llvm/test/Transforms/ObjCARC/basic.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D115302.394068.patch
Type: text/x-patch
Size: 9874 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211213/7a7ffd84/attachment.bin>


More information about the llvm-commits mailing list