[PATCH] D86053: [APInt] Allow self-assignment with libstdc++

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka added a comment.

I would prefer to remove if def completly and always compile with  if (this == &that)
Original patch justified removing of (this == &that) to save 4k. I don't think it's worth it.
However if want to remove this from release maybe we should do the following, as EXPENSIVE_CHECKS is the one who triggers std::shuffle in llvm::sort?

  #ifdef EXPENSIVE_CHECKS
     if (this == &that)
        return *this;
  #endif


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86053