[PATCH] D63569: Make TargetParserTest.ARMExtensionFeatures not run out of memory on 32-bit (PR42316)

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 20 01:46:03 PDT 2019


dmgreen accepted this revision.
dmgreen added a comment.
This revision is now accepted and ready to land.

I agree that the old test wasn't very helpful and this one is a lot better. Can you also check that AEK_INVALID returns no features, and the others arm calls only add one (size() == 1).

Other than that, LGTM.


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

https://reviews.llvm.org/D63569





More information about the llvm-commits mailing list