[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