[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