[PATCH] D51362: [GlobalISel][IRTranslator] Canonicalize G_ICMP to have constant operands last

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 5 08:22:44 PDT 2018

Previous message: [llvm] r341477 - [XRay] Use `const` instead of `constexpr` in storage declaration
Next message: [llvm] r341478 - [Hexagon] Ignore unnamed globals in HexagonConstExtenders
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

qcolombet added a comment.

> If we agree that an input canonicalization pass is needed then doing this there adds much less complexity than teaching tablegen that G_ICMPs are special (or somehow adding legalisation rules to tablegen?).

Agree.


Repository:
  rL LLVM

https://reviews.llvm.org/D51362

Previous message: [llvm] r341477 - [XRay] Use `const` instead of `constexpr` in storage declaration
Next message: [llvm] r341478 - [Hexagon] Ignore unnamed globals in HexagonConstExtenders
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list