[PATCH] D141777: [X86] Reenable store merging post-legalization

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ISelLowering.h:1072-1073
+                                      StoreSource StoreSrc) const override {
+      return StoreSrc == StoreSource::Unknown ||
+             StoreSrc != StoreSource::ExtractSubvector;
     }
----------------
This is a bit ugly. I'm using `StoreSource::Unknown` as
"we are post-legalization. do we allow merging *any* sources?".
Is that too ugly?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141777