[PATCH] D62394: [ARM][CMSE] Add CMSE header & builtins

2019-10-06 Thread Momchil Velikov via Phabricator via cfe-commits
chill added a comment. I hope I will be able to pick this up in the following weeks and land patches a couple of weeks later. Sorry for the delay, but priorities shift all the time ;) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62394/new/ https://reviews.llvm.org/D62394 __

[PATCH] D62394: [ARM][CMSE] Add CMSE header & builtins

2019-09-18 Thread Pablo Barrio via Phabricator via cfe-commits
pbarrio added a comment. Hi, CMSE upstreaming is indeed one of our priorities. So yes, we are very interested in your feedback. And no, CMSE upstreaming is not abandoned, just going a bit slow ATM :( but any help reviewing is much appreciated! :) @chill would it make sense to start upstream oth

[PATCH] D62394: [ARM][CMSE] Add CMSE header & builtins

2019-09-17 Thread Sigvart Hovland via Phabricator via cfe-commits
sigvartmh resigned from this revision. sigvartmh added a comment. Not used to phabricator tried to remove my request for change. So no one is working on upstreaming CMSE to LLVM now? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62394/new/ https://reviews.llvm.org/D62394 __

[PATCH] D62394: [ARM][CMSE] Add CMSE header & builtins

2019-09-17 Thread Dave Green via Phabricator via cfe-commits
dmgreen added a subscriber: chill. dmgreen added a comment. I'm afraid the upstreaming of CMSE has stalled, and this is not all that would be needed to get it working. This adds some header files and clang builtins, the selection of them in the backend isn't yet present, hence the error you are

[PATCH] D62394: [ARM][CMSE] Add CMSE header & builtins

2019-09-17 Thread Sigvart Hovland via Phabricator via cfe-commits
sigvartmh added a comment. In D62394#1672570 , @javed.absar wrote: > In D62394#1672551 , @sigvartmh wrote: > > > Maybe I'm doing something wrong tried to apply these patches but when > > trying to build code which

[PATCH] D62394: [ARM][CMSE] Add CMSE header & builtins

2019-09-17 Thread Javed Absar via Phabricator via cfe-commits
javed.absar added a comment. In D62394#1672551 , @sigvartmh wrote: > Maybe I'm doing something wrong tried to apply these patches but when trying > to build code which uses cmse I get > > Cannot select: intrinsic %llvm.arm.cmse.tt > fatal error: error in

[PATCH] D62394: [ARM][CMSE] Add CMSE header & builtins

2019-09-17 Thread Sigvart Hovland via Phabricator via cfe-commits
sigvartmh requested changes to this revision. sigvartmh added a comment. This revision now requires changes to proceed. Maybe I'm doing something wrong tried to apply these patches but when trying to build code which uses cmse I get Cannot select: intrinsic %llvm.arm.cmse.tt fatal error: error i

[PATCH] D62394: [ARM][CMSE] Add CMSE header & builtins

2019-05-24 Thread Javed Absar via Phabricator via cfe-commits
javed.absar created this revision. javed.absar added reviewers: snidertm, dmgreen. Herald added subscribers: kristof.beyls, mgorny. This is patch C2 as mentioned in RFC http://lists.llvm.org/pipermail/cfe-dev/2019-March/061834.html This adds cmse builtin functions, and introduces arm_cmse.h head