[PATCH] D102295: [FunctionAttr][Attributor] Cleanup `nosync` checks of memory intrinsics

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 18 21:09:30 PDT 2021


nhaehnle updated this revision to Diff 346337.
nhaehnle edited the summary of this revision.
nhaehnle added a comment.
Herald added subscribers: kerbowa, jvesely.

Invert the logic of the patch: mem intrinsics shouldn't be unconditionally marked `nosync` in the first place.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102295

Files:
  llvm/include/llvm/IR/Intrinsics.td
  llvm/test/Analysis/BasicAA/cs-cs.ll
  llvm/test/Analysis/TypeBasedAliasAnalysis/functionattrs.ll
  llvm/test/CodeGen/AMDGPU/addrspacecast-constantexpr.ll
  llvm/test/Transforms/Attributor/ArgumentPromotion/X86/attributes.ll
  llvm/test/Transforms/Attributor/ArgumentPromotion/X86/min-legal-vector-width.ll
  llvm/test/Transforms/Attributor/heap_to_stack.ll
  llvm/test/Transforms/Attributor/lowerheap.ll
  llvm/test/Transforms/Attributor/nosync.ll
  llvm/test/Transforms/DeadStoreElimination/multiblock-malloc-free.ll
  llvm/test/Transforms/MemCpyOpt/memcpy.ll
  llvm/test/Transforms/ObjCARC/nested.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D102295.346337.patch
Type: text/x-patch
Size: 23414 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210519/464b684f/attachment.bin>


More information about the llvm-commits mailing list