[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