[PATCH] D85047: [Hexagon] Use InstSimplify instead of ConstantProp

Krzysztof Parzyszek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 1 09:58:22 PDT 2020


kparzysz added a comment.

These failures seem to be caused by optimizing the code "too much", and simply removing the parts that the tests were looking for.

Could you add a debug flag in HexagonTargetMachine.cpp, and add the simplification pass conditionally, similarly to how line 322 does it?  The pass would be added by default, but these failing tests could exclude it (see the `-hexagon-initial-cfg-cleanup=0` in some of the failing tests, that's the exact same problem caused by some other change in optimization passes).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85047



More information about the llvm-commits mailing list