[PATCH] D61169: [CodeGenPrepare] Don't split the store if it is volatile

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 28 23:09:47 PDT 2019


dmgreen added a comment.

Looks like there is similar code in DAGCombiner that may need a volatile check too, if you search for isMultiStoresCheaperThanBitsMerge.



================
Comment at: llvm/test/CodeGen/PowerPC/splitstore-check-volatile.ll:3
+;
+; RUN: llc -mtriple=powerpc64le-unknown-linux-gnu -stop-after codegenprepare -force-split-store < %s  | FileCheck %s
+
----------------
You could just make this an opt test, running the codegenprepare pass.


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

https://reviews.llvm.org/D61169





More information about the llvm-commits mailing list