Fwd: [PATCH] [ARM64] Enable alignment control option in front-end for ARM64 - Clang.

Kevin Qin kevinqindev at gmail.com
Thu Apr 24 00:31:01 PDT 2014


I modified the CC list to add cfe-commit and llvm-commit, but it seems
doesn't work. So I manually forward this email. Sorry for any
inconvenience.

---------- Forwarded message ----------
From: Kevin Qin <kevinqindev at gmail.com>
Date: 2014-04-23 18:05 GMT+08:00
Subject: [PATCH] [ARM64] Enable alignment control option in front-end for
ARM64 - Clang.
To: kevinqindev at gmail.com, t.p.northover at gmail.com
Cc: amara.emerson at arm.com


Hi t.p.northover,

Hi Tim and other reviewers,

This patch is to get "-mno-unaligned-access" and "-munaligned-access" work
in front-end for ARM64 target.

One thing worth to mention is, as ARM and ARM64 shared same command option
in front end, does it need to share same command option in backend? If we
decide to do this, the opt option "arm-strict-align" and
"arm64-strict-align" would be combined to a single opt
option("arm-strict-align" for example) which is shared in two backends. My
question is, is it necessary to do this? And if we want do this, how to
share opt object between two backends in a good way? If there is any other
idea around this, please let me know. Thanks.

http://reviews.llvm.org/D3466

Files:
  include/clang/Driver/Options.td
  lib/Driver/Tools.cpp
  test/Driver/arm-alignment.c



-- 
Best Regards,

Kevin Qin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140424/6d466eb1/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D3466.8767.patch
Type: text/x-patch
Size: 5463 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140424/6d466eb1/attachment.bin>


More information about the cfe-commits mailing list