[PATCH] D61273: [globalisel] Update the legalizer documentation

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 29 12:04:44 PDT 2019


dsanders created this revision.
dsanders added reviewers: bogner, aditya_nandakumar, volkan, aemerson, paquette, arsenm.
Herald added subscribers: Petar.Avramovic, jfb, kristof.beyls, rovka, wdng.
Herald added a project: LLVM.

- The getActionDefinitionsBuilder() is now documented.
  - Includes descriptions of the various actions (legal*, widenScalar*, lower*, etc).
  - Includes descriptions of the various predicates (*If, *For, *ForCartesianProduct, etc.)
  - Includes the rule-order details
- Removed the out-of-date prohibition on non-power-of-2 types.
- Removed the Vector types section since it was incorrect and vectors follow the same ruleset as scalars. They're only special in the sense that more of the actions and predicates are meaningful for them (e.g. moreElements).
- Clarified the position on context sensitive legality (which is not permitted) and contrasted this with context sensitive legalization (which is permitted).


Repository:
  rL LLVM

https://reviews.llvm.org/D61273

Files:
  docs/GlobalISel.rst

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61273.197156.patch
Type: text/x-patch
Size: 9913 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190429/304af5e2/attachment.bin>


More information about the llvm-commits mailing list