[PATCH] D83029: [SVE] Fix invalid assert in expand_DestructiveOp.

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 2 04:48:10 PDT 2020


paulwalker-arm marked an inline comment as done.
paulwalker-arm added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ExpandPseudoInsts.cpp:481
   if (FalseZero) {
+#ifndef NDEBUG
+    assert(DOPRegIsUnique && "The destructive operand should be unique");
----------------
sdesmalen wrote:
> Why are you wrapping this in `#ifndef NDEBUG` ?
The definition of DOPRegIsUnique is protected by NDEBUG.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83029





More information about the llvm-commits mailing list