This is the first in a set of patches to add AAP as an experimental target to LLVM. The only changes this patch makes is to recognize the 'aap' triple.
Future patches will incrementally add the remainder of the backend.
Paths
| Differential D23664
[AAP] (1) Add AAP target triple AbandonedPublic Authored by edward-jones on Aug 18 2016, 8:36 AM.
Details
Diff Detail Event Timelineedward-jones updated this object. Comment Actions This looks good to me, but I'll let other people review and approve, since I was the only one on the thread. :) Comment Actions 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. This revision is now accepted and ready to land.Aug 19 2016, 4:42 AM
Revision Contents
Diff 100978 include/llvm/ADT/Triple.h
lib/Support/Triple.cpp
unittests/ADT/TripleTest.cpp
|
Would be nice to sort these.