[codeview] Use push_macro to avoid conflicts instead of a prefix
authorReid Kleckner <rnk@google.com>
Thu, 16 Aug 2018 17:34:31 +0000 (17:34 +0000)
committerReid Kleckner <rnk@google.com>
Thu, 16 Aug 2018 17:34:31 +0000 (17:34 +0000)
commitbd5d71229db6c6c4f2446229de9b0e930b7f6be9
treec7bf8f171a38310c196d7ad99ad68cd029d52850
parenteb189a0ef77b6399b822de2a4e8847e573c50702
[codeview] Use push_macro to avoid conflicts instead of a prefix

Summary:
This prefix was added in r333421, and it changed our dumper output to
say things like "CVRegEAX" instead of just "EAX". That's a functional
change that I'd rather avoid.

I tested GCC, Clang, and MSVC, and all of them support #pragma
push_macro. They don't issue warnings whem the macro is not defined
either.

I don't have a Mac so I can't test the real termios.h header, but I
looked at the termios.h sources online and looked for other conflicts.
I saw only the CR* macros, so those are the ones we work around.

Reviewers: zturner, JDevlieghere

Subscribers: hiraditya, llvm-commits

Differential Revision: https://reviews.llvm.org/D50851

llvm-svn: 339907
14 files changed:
lld/test/COFF/pdb-type-server-simple.test
llvm/include/llvm/DebugInfo/CodeView/CodeViewRegisters.def
llvm/lib/DebugInfo/PDB/Native/NativeRawSymbol.cpp
llvm/lib/Target/X86/MCTargetDesc/X86MCTargetDesc.cpp
llvm/test/DebugInfo/COFF/fp-stack.ll
llvm/test/DebugInfo/COFF/local-variable-gap.ll
llvm/test/DebugInfo/COFF/local-variables.ll
llvm/test/DebugInfo/COFF/pieces.ll
llvm/test/DebugInfo/COFF/register-variables.ll
llvm/test/DebugInfo/COFF/types-array.ll
llvm/test/DebugInfo/PDB/DIA/pdbdump-symbol-format.test
llvm/test/DebugInfo/X86/dbg-declare-inalloca.ll
llvm/test/MC/COFF/cv-def-range-gap.s
llvm/unittests/DebugInfo/CodeView/TypeIndexDiscoveryTest.cpp