[PATCH] D68263: [NFC][InstCombine] Add tests that show a number of canonicalization opportunities

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: llvm/test/Transforms/InstCombine/phi-equal-incoming-pointers.ll:2-5
+; RUN: opt -passes=instcombine,verify -S < %s | FileCheck %s
+
+; Make sure GVN won't undo the transformation:
+; RUN: opt -passes=instcombine,gvn -S < %s | FileCheck --check-prefix=CHECK-AFTER-GVN %s
----------------
In general depending on more than one pass in tests is frowned upon.
Does 
```
; RUN: opt -passes=instcombine,verify -S < %s | FileCheck %s --check-prefixes=ALL,INSTCOMBINE

; Make sure GVN won't undo the transformation:
; RUN: opt -passes=instcombine,gvn -S < %s | FileCheck --check-prefixes=ALL,INSTCOMBINEGVN %s
```
get rid of dedicated `; CHECK-AFTER-GVN` check lines?


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

https://reviews.llvm.org/D68263