[PATCH] D84201: [LangRef] Add integer wrapping operand bundles.
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 21 07:32:00 PDT 2020
fhahn added a comment.
In D84201#2164274 <https://reviews.llvm.org/D84201#2164274>, @arsenm wrote:
> In D84201#2164166 <https://reviews.llvm.org/D84201#2164166>, @fhahn wrote:
>
> > 2. adding the flags directly to calls means all calls need to pay some cost (a few extra bits somewhere)
>
>
> But calls already have fast math flags; I assume these would alias the same bits
Oh right, I guess we could do that and then it would be free in terms of size. But I think the main question is if nuw/nsw flags are best use for those bits and what makes them special, as compared to other properties for which we have bundles or call site attributes (mainly point 3) about the general usefulness of those flags.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D84201/new/
https://reviews.llvm.org/D84201
More information about the llvm-commits
mailing list