[PATCH] D83833: [GISel] Add new GISel combiners for G_SELECT

Michael Kitzan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 15 12:05:19 PDT 2020

Previous message: [PATCH] D83833: [GISel] Add new GISel combiners for G_SELECT
Next message: [llvm] f49edaf - Fix test that was accidentally adding the llvm-as binary into an
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

mkitzan added a comment.

Maybe? It's unclear to me what makes a `ConstantExpr` poison in the first place, and how combining with it would result in incorrect code (...new hire rolling through).


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

https://reviews.llvm.org/D83833

Previous message: [PATCH] D83833: [GISel] Add new GISel combiners for G_SELECT
Next message: [llvm] f49edaf - Fix test that was accidentally adding the llvm-as binary into an
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list