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

Simon Atanasyan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 23 01:13:17 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] D50578: [DAGCombiner][Mips] Don't combine bitcast+store after LegalOperations when the store is volatile, if the resulting store isn't Legal
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

atanasyan accepted this revision.
atanasyan added a comment.
This revision is now accepted and ready to land.

LGTM
MIPS code becomes a bit longer, but I think it can be fixed in the future.


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] D50578: [DAGCombiner][Mips] Don't combine bitcast+store after LegalOperations when the store is volatile, if the resulting store isn't Legal
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list