[PATCH] D93189: [WIP] Introduce the `!nocapture` metadata and "nocapture_use" operand bundle

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 14 00:41:23 PST 2020


nikic added inline comments.


================
Comment at: llvm/lib/Analysis/CaptureTracking.cpp:321
           return;
       }
       break;
----------------
jdoerfert wrote:
> Interestingly, we seem to not consider operand bundle uses to be capturing. This is (IMHO) wrong in general. "nocapure_use" does not capture but others should be assumed to do. Will propose a separate patch at some point.
Data operands = Arguments + operand bundles, so this should also consider operand bundle uses.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93189



More information about the llvm-commits mailing list