[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