[PATCH] D11470: [ARM] Define subtarget feature "strict-align"

Derek Schuff dschuff at google.com
Thu Jul 23 13:12:40 PDT 2015


dschuff added a comment.

I don't have a strong opinion on whether this should be a subtarget feature vs a backend option (although a subtarget feature does seem right to me).
But otherwise this looks OK from the NaCl perspective. The clang patch looks like it has the right behavior wrt nacl-clang (i.e. the non-PNaCl clang toolchain), and it shouldn't be a problem for us to fix our out-of-tree pnacl-translate tool.


http://reviews.llvm.org/D11470







More information about the llvm-commits mailing list