[PATCH] D116221: [AArch64][ARM][Clang] Unaligned Access Warning Added

2022-01-20 Thread Pablo Barrio via Phabricator via cfe-commits
pbarrio accepted this revision. pbarrio added a comment. This revision is now accepted and ready to land. LGTM too. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116221/new/ https://reviews.llvm.org/D116221 ___ cfe-commits mailing list

[PATCH] D116221: [AArch64][ARM][Clang] Unaligned Access Warning Added

2021-12-23 Thread Pablo Barrio via Phabricator via cfe-commits
pbarrio requested changes to this revision. pbarrio added a comment. This revision now requires changes to proceed. Suggest clarifying the commit message to something like: "Added warning for potential cases of unaligned access when option -mno-unaligned-access has been specified". Nice

[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

[PATCH] D57060: [NFC][Clang] Add driver tests for sb and predres

2019-01-25 Thread Pablo Barrio via Phabricator via cfe-commits
pbarrio accepted this revision. pbarrio added a comment. This revision is now accepted and ready to land. Same idea as https://reviews.llvm.org/D54961 but for two other command line options. Approved! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57060/new/

[PATCH] D54961: [AArch64] Add command-line option for SSBS

2018-11-27 Thread Pablo Barrio via Phabricator via cfe-commits
pbarrio created this revision. pbarrio added reviewers: olista01, samparker, aemerson. Herald added subscribers: kristof.beyls, javed.absar. SSBS (Speculative Store Bypass Safe) is only mandatory from 8.5 onwards but is optional from Armv8.0-A. This patch adds testing for the ssbs command line