diff options
| author | Nick Desaulniers <ndesaulniers@google.com> | 2019-04-26 18:45:04 +0000 |
|---|---|---|
| committer | Nick Desaulniers <ndesaulniers@google.com> | 2019-04-26 18:45:04 +0000 |
| commit | 7ab164c4a427b559a7a47fa62ef365862705f950 (patch) | |
| tree | c8aacfde9e16dcbd85708bf2424ce6f9c94b76aa /llvm/lib/Target/X86/X86AsmPrinter.h | |
| parent | 74967cb4e04c9899601b8c84901e99e797d49135 (diff) | |
| download | bcm5719-llvm-7ab164c4a427b559a7a47fa62ef365862705f950.tar.gz bcm5719-llvm-7ab164c4a427b559a7a47fa62ef365862705f950.zip | |
[AsmPrinter] refactor to support %c w/ GlobalAddress'
Summary:
Targets like ARM, MSP430, PPC, and SystemZ have complex behavior when
printing the address of a MachineOperand::MO_GlobalAddress. Move that
handling into a new overriden method in each base class. A virtual
method was added to the base class for handling the generic case.
Refactors a few subclasses to support the target independent %a, %c, and
%n.
The patch also contains small cleanups for AVRAsmPrinter and
SystemZAsmPrinter.
It seems that NVPTXTargetLowering is possibly missing some logic to
transform GlobalAddressSDNodes for
TargetLowering::LowerAsmOperandForConstraint to handle with "i" extended
inline assembly asm constraints.
Fixes:
- https://bugs.llvm.org/show_bug.cgi?id=41402
- https://github.com/ClangBuiltLinux/linux/issues/449
Reviewers: echristo, void
Reviewed By: void
Subscribers: void, craig.topper, jholewinski, dschuff, jyknight, dylanmckay, sdardis, nemanjai, javed.absar, sbc100, jgravelle-google, eraman, kristof.beyls, hiraditya, aheejin, kbarton, fedor.sergeev, jrtc27, atanasyan, jsji, llvm-commits, kees, tpimh, nathanchance, peter.smith, srhines
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D60887
llvm-svn: 359337
Diffstat (limited to 'llvm/lib/Target/X86/X86AsmPrinter.h')
| -rw-r--r-- | llvm/lib/Target/X86/X86AsmPrinter.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Target/X86/X86AsmPrinter.h b/llvm/lib/Target/X86/X86AsmPrinter.h index fd49d154c65..a011310970b 100644 --- a/llvm/lib/Target/X86/X86AsmPrinter.h +++ b/llvm/lib/Target/X86/X86AsmPrinter.h @@ -102,7 +102,7 @@ class LLVM_LIBRARY_VISIBILITY X86AsmPrinter : public AsmPrinter { // Choose between emitting .seh_ directives and .cv_fpo_ directives. void EmitSEHInstruction(const MachineInstr *MI); - void PrintSymbolOperand(const MachineOperand &MO, raw_ostream &O); + void PrintSymbolOperand(const MachineOperand &MO, raw_ostream &O) override; void PrintOperand(const MachineInstr *MI, unsigned OpNo, raw_ostream &O); void PrintModifiedOperand(const MachineInstr *MI, unsigned OpNo, raw_ostream &O, const char *Modifier); |

