[PATCH] D48223: DAG combine "and|or (select c, -1, 0), x" -> "select c, x, 0|-1"
    Stanislav Mekhanoshin via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Sat Jun 16 01:00:59 PDT 2018
    
        - Previous message: [PATCH] D48223: DAG combine "and|or (select c, -1, 0), x" -> "select c, x, 0|-1"
- Next message: [PATCH] D48223: DAG combine "and|or (select c, -1, 0), x" -> "select c, x, 0|-1"
-  Messages sorted by: 
              [ date ]
              [ thread ]
              [ subject ]
              [ author ]
         
  
rampitec added a comment.
In https://reviews.llvm.org/D48223#1134457, @lebedev.ri wrote:
> Would be good to have additional small test for some other target (e.g. x86), too.
Sure. It turns out x86 needs it even more probably. I have created the test and it turns out both andl and orl were prevent before the patch.
https://reviews.llvm.org/D48223
    
    
        
	- Previous message: [PATCH] D48223: DAG combine "and|or (select c, -1, 0), x" -> "select c, x, 0|-1"
- Next message: [PATCH] D48223: DAG combine "and|or (select c, -1, 0), x" -> "select c, x, 0|-1"
-  Messages sorted by: 
              [ date ]
              [ thread ]
              [ subject ]
              [ author ]
         
More information about the llvm-commits
mailing list