[funcattrs] Infer nosync from instruction walk
authorPhilip Reames <listmail@philipreames.com>
Thu, 8 Apr 2021 21:02:39 +0000 (14:02 -0700)
committerPhilip Reames <listmail@philipreames.com>
Thu, 8 Apr 2021 21:05:00 +0000 (14:05 -0700)
commit35393c865c2cb1afbf1315a9d48ac41994cd1344
treee184cbf5e65d45d55c7e6d11848fd755ae230917
parentc5d1ccbcdfb11c29c89a2789ac1e89cf4b270be7
[funcattrs] Infer nosync from instruction walk

Pretty straightforward use of existing infrastructure and port of the attributor inference rules for nosync.

A couple points of interest:
* I deliberately switched from "monotonic or better" to "unordered or better". This is simply me being conservative and is better in line with the rest of the optimizer. We treat monotonic conservatively pretty much everywhere.
* The operand bundle test change is suspicious. It looks like we might have missed something here, but if so, it's an issue with the existing nofree inference as well. I'm going to take a closer look at that separately.
* I needed to keep the previous inference from readnone. This surprised me, but made sense once I realized readonly inference goes to lengths to reason about local vs non-local memory and that writes to local memory are okay. This is fine for the purpose of nosync, but would e.g. prevent us from inferring nofree from readnone - which is slightly surprising.

Differential Revision: https://reviews.llvm.org/D99769
llvm/lib/Transforms/IPO/FunctionAttrs.cpp
llvm/test/Analysis/TypeBasedAliasAnalysis/functionattrs.ll
llvm/test/CodeGen/AMDGPU/inline-attr.ll
llvm/test/Transforms/FunctionAttrs/nofree.ll
llvm/test/Transforms/FunctionAttrs/nosync.ll
llvm/test/Transforms/FunctionAttrs/operand-bundles-scc.ll
llvm/test/Transforms/FunctionAttrs/read-write-scc.ll
llvm/test/Transforms/InferFunctionAttrs/norecurse_debug.ll