[PATCH] D27636: [NVPTX] Move PropertyAnnotationNames into a cc file.

Justin Lebar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 9 15:57:23 PST 2016


jlebar created this revision.
jlebar added a reviewer: tra.
jlebar added a subscriber: llvm-commits.
Herald added subscribers: mgorny, jholewinski.

Previously they were defined as a 2D char array in a header file.  This
is kind of overkill -- we can let the linker lay out these strings
however it pleases.  While we're at it, we might as well move these
chars into a cpp file so we're not playing with ODR-risky code.


https://reviews.llvm.org/D27636

Files:
  llvm/lib/Target/NVPTX/MCTargetDesc/CMakeLists.txt
  llvm/lib/Target/NVPTX/MCTargetDesc/NVPTXBaseInfo.cpp
  llvm/lib/Target/NVPTX/MCTargetDesc/NVPTXBaseInfo.h
  llvm/lib/Target/NVPTX/NVPTXUtilities.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27636.80966.patch
Type: text/x-patch
Size: 10533 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161209/38b10b29/attachment.bin>


More information about the llvm-commits mailing list