[PATCH] D98908: Update basic deref API to account for possiblity of free [NFC]

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 18 17:13:56 PDT 2021


reames created this revision.
reames added reviewers: jdoerfert, apilipenko, nlopes.
Herald added subscribers: dexonsmith, okura, kuter, dantrushin, hiraditya, mcrosier.
Herald added a reviewer: bollu.
reames requested review of this revision.
Herald added a reviewer: sstefan1.
Herald added a reviewer: baziotis.
Herald added a subscriber: bbn.
Herald added a project: LLVM.

This patch is plumbing to support work towards the goal outlined in the recent llvm-dev post "[llvm-dev] RFC: Decomposing deref(N) into deref(N) + nofree".

The point of this change is purely to simplify iteration on other pieces on way to making the switch.  Rebuilding with a change to Value.h is slow and painful, so I want to get the API change landed.  Once that's done, I plan to more closely audit each caller, add the inference rules in their own patch, then post a patch with the langref changes and test diffs.  The value of the command line flag is that we can exercise the inference logic in standalone patches without needing the whole switch ready to go just yet.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D98908

Files:
  llvm/include/llvm/Analysis/MemoryBuiltins.h
  llvm/include/llvm/IR/Value.h
  llvm/lib/Analysis/BasicAliasAnalysis.cpp
  llvm/lib/Analysis/CaptureTracking.cpp
  llvm/lib/Analysis/Loads.cpp
  llvm/lib/IR/Value.cpp
  llvm/lib/Transforms/IPO/AttributorAttributes.cpp
  llvm/lib/Transforms/InstCombine/InstCombineCasts.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D98908.331727.patch
Type: text/x-patch
Size: 8218 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210319/b17aebbc/attachment.bin>


More information about the llvm-commits mailing list