[PATCH] D129765: [DAG] SimplifyDemandedBits - don't early-out for multiple use values (WIP)

Sam Elliott via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 14 07:01:19 PDT 2022


lenary added a comment.

> @lenary Any thoughts on what is going so wrong in aea-erratum-fix.ll? SimplifyDemandedBits does tend to mess with specific patterns that this might be depending on.

A quick look suggests you shouldn't be worried about it. I need to revisit the tests to simplify them, because the test is just checking the inputs to the aese/aesd instructions are from particular instructions, rather than all the prior logic, and I had failed to notice how complex that test had become.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129765



More information about the llvm-commits mailing list