This is an archive of the discontinued LLVM Phabricator instance.

Allow armv{7,7s,7k,7m,7em} builds
ClosedPublic

Authored by jroelofs on May 23 2017, 2:02 PM.

Details

Diff Detail

Event Timeline

jroelofs created this revision.May 23 2017, 2:02 PM
compnerd accepted this revision.May 23 2017, 5:26 PM

Modulo the IT block macros, LGTM.

cmake/builtin-config-ix.cmake
27

Probably should add armv7 as well.

lib/builtins/arm/aeabi_cdcmp.S
103

We have macros for the IT blocks, can you use those, please? Similar throughout.

This revision is now accepted and ready to land.May 23 2017, 5:26 PM
jroelofs updated this revision to Diff 100101.May 24 2017, 8:42 AM
jroelofs retitled this revision from Allow armv7m and armv7em builds to Allow armv{7,7s,7k,7m,7em} builds.

implement feedback

jroelofs closed this revision.May 24 2017, 8:53 AM

r303765