[PATCH] D79391: [DSE] Remove noop stores in MSSA.

Zoe Carver via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 29 23:22:34 PDT 2020


zoecarver added a comment.

@fhahn I really appreciate all your help with this patch. The tests are updated.

---

> Hm I think cmake might not pick up CXXFLAGS...

I was passing the flags to the llvm-test-suite build but, I also tried when building the mono-repo. I tried both with `CXXFLAGS` and with `DCMAKE_C_FLAGS` as suggested and made sure to export `EnableMemorySSA=1` (and even tried a clean build of both) but I still get the same error when running `utils/compare.py`. I'll take another look at the docs and keep playing around with it to see if I can get it working tomorrow.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79391





More information about the llvm-commits mailing list