[PATCH] D11845: Properly pass through the PIC mode to the integrated assembler when doing assembly-only, and unify the Driver's PIC argument parsing.

James Y Knight via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 7 13:21:21 PDT 2015


jyknight created this revision.
jyknight added reviewers: rnk, echristo.
jyknight added a subscriber: cfe-commits.
Herald added subscribers: srhines, danalbert, tberghammer.

On a few architectures, parsing of assembly files annoyingly depends
on whether PIC is enabled or not. This was handled for external 'as'
already (passing -KPIC), but was missed for calls to the standalone
internal assembler.

The integrated-as.s test needed to be modified to not expect
-fsanitize=address to be unused, as now fsanitize *IS* used for
assembly, since -fsanitize=memory can sometimes imply -fPIE, which the
assembler needs to know (gack!!).

http://reviews.llvm.org/D11845

Files:
  include/clang/Driver/CC1Options.td
  lib/Driver/Tools.cpp
  test/Driver/integrated-as.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11845.31540.patch
Type: text/x-patch
Size: 24204 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150807/dea2d0d6/attachment-0001.bin>


More information about the cfe-commits mailing list