[PATCH] D71739: [AssumeBundles] Use operand bundles to encode alignment assumptions

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 22 10:43:29 PDT 2020


rjmccall added a comment.

> there is any IR legality restriction about what combination or number of bundles can be present in an assume. however there is restrictions about argument of a given bundle.
>  having a single "align" bundle is just how the front-end used assume bundles for its alignment assumptions.

Hmm.  It does seem to be true that operand bundles don't inherently enforce that the names are unique on a callsite; the verifier just has an ever-growing list of bundles that it does that for specially.  That feels weird to me, but alright.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71739





More information about the cfe-commits mailing list