[PATCH] D81472: [Matrix] Update load/store intrinsics.
Stefan Stipanovic via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 11 00:37:38 PDT 2020
sstefan1 added inline comments.
================
Comment at: llvm/include/llvm/IR/Intrinsics.td:1465
+ WriteOnly<ArgIndex<1>>, ImmArg<ArgIndex<3>>,
+ ImmArg<ArgIndex<4>>, ImmArg<ArgIndex<5>>]>;
----------------
jdoerfert wrote:
> fhahn wrote:
> > jdoerfert wrote:
> > > [Drive by][unrelated] I think we should add `nocapture` to the ptr argument and `nosync` to all of them (until we have the white/blacklist for intrinsics with sensible defaults).
> > Thanks for pointing that out. There are just too many attributes to keep track of. I wish we had some kind of attribute 'group' to say: just reads/write from the pointer, no capture and other stuff
> We (@sstefan1) proposed a whitelist and blacklist approach for intrinsics before. Hasn't gone anywhere yet. For the OpenMP runtime functions we actually have such attribute groups. Either way is better than what we do so far.
I will try to get back to that soon.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D81472/new/
https://reviews.llvm.org/D81472
More information about the llvm-commits
mailing list