[PATCH] D50578: [DAGCombiner][Mips] Don't combine bitcast+store after LegalOperations when the store is volatile, if the resulting store isn't Legal

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 10 16:19:40 PDT 2018

Previous message: [PATCH] D50578: [DAGCombiner][Mips] Don't combine bitcast+store after LegalOperations when the store is volatile, if the resulting store isn't Legal
Next message: [PATCH] D50579: [LV] Teach about non header phis that have uses outside the loop
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

efriedma added a comment.

Code change LGTM, but someone familiar with the MIPS backend should review the test changes.


https://reviews.llvm.org/D50578

Previous message: [PATCH] D50578: [DAGCombiner][Mips] Don't combine bitcast+store after LegalOperations when the store is volatile, if the resulting store isn't Legal
Next message: [PATCH] D50579: [LV] Teach about non header phis that have uses outside the loop
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list