[PATCH] D17183: Make TargetInfo store an actual DataLayout instead of a string.

James Y Knight via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 11 18:37:30 PST 2016


jyknight created this revision.
jyknight added a reviewer: echristo.
jyknight added a subscriber: cfe-commits.

Use it to calculate UserLabelPrefix, instead of specifying it (often
incorrectly).

Note that the *actual* user label prefix has always come from the
DataLayout, and is handled within LLVM. The main thing clang's
TargetInfo::UserLabelPrefix did was to set the #define value. Having
these be different from each-other is just silly.

http://reviews.llvm.org/D17183

Files:
  include/clang/Basic/TargetInfo.h
  include/clang/CodeGen/BackendUtil.h
  lib/AST/Mangle.cpp
  lib/Basic/TargetInfo.cpp
  lib/Basic/Targets.cpp
  lib/CodeGen/BackendUtil.cpp
  lib/CodeGen/CodeGenAction.cpp
  lib/CodeGen/ModuleBuilder.cpp
  lib/CodeGen/ObjectFilePCHContainerOperations.cpp
  lib/Frontend/InitPreprocessor.cpp
  test/Preprocessor/init.c
  tools/libclang/CIndex.cpp
  unittests/Basic/SourceManagerTest.cpp
  unittests/Lex/LexerTest.cpp
  unittests/Lex/PPConditionalDirectiveRecordTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17183.47763.patch
Type: text/x-patch
Size: 55700 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160212/44692056/attachment-0001.bin>


More information about the cfe-commits mailing list