[PATCH] D23664: [AAP] (1) Add AAP target triple

Alex Bradbury via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 19 04:42:05 PDT 2016


asb added a subscriber: asb.
asb accepted this revision.
asb added a reviewer: asb.
asb added a comment.
This revision is now accepted and ready to land.

LGTM. The target triple is added everywhere appropriate and the tests are sensible. It's a minor annoyance that the relevant enum and case statements are inconsistent about whether they're in alphabetic order or order of addition of that backend, but that's not AAP's fault and your additions are in sensible places.


https://reviews.llvm.org/D23664





More information about the llvm-commits mailing list