[PATCH] D36331: Add ARC backend

Pete Couperus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 7 11:07:40 PDT 2017


petecoup added inline comments.


================
Comment at: llvm/lib/Target/ARC/ARCExpandPseudos.cpp:97
+  }
+  return ToRemove.size() > 0;
+}
----------------
Eugene.Zelenko wrote:
> arsenm wrote:
> > .empty()
> This what Clang-tidy readability-container-size-empty should catch. I suggested such run before.
Hmm, that's odd, sorry about that.  I will redo these checks, and address the other comments from here.



https://reviews.llvm.org/D36331





More information about the llvm-commits mailing list