forked from OSchip/llvm-project
[ObjC] Use consistent comment style in inline asm
The comment markers accepted by the assembler vary between different targets, but '//' is always accepted, so we should use that for consistency. Differential revision: https://reviews.llvm.org/D36666 llvm-svn: 311325
This commit is contained in:
parent
b09bd74da8
commit
7f18864473
|
@ -1085,7 +1085,7 @@ public:
|
|||
|
||||
StringRef getARCRetainAutoreleasedReturnValueMarker() const override {
|
||||
return "movl\t%ebp, %ebp"
|
||||
"\t\t## marker for objc_retainAutoreleaseReturnValue";
|
||||
"\t\t// marker for objc_retainAutoreleaseReturnValue";
|
||||
}
|
||||
};
|
||||
|
||||
|
@ -4880,7 +4880,7 @@ public:
|
|||
: TargetCodeGenInfo(new AArch64ABIInfo(CGT, Kind)) {}
|
||||
|
||||
StringRef getARCRetainAutoreleasedReturnValueMarker() const override {
|
||||
return "mov\tfp, fp\t\t# marker for objc_retainAutoreleaseReturnValue";
|
||||
return "mov\tfp, fp\t\t// marker for objc_retainAutoreleaseReturnValue";
|
||||
}
|
||||
|
||||
int getDwarfEHStackPointer(CodeGen::CodeGenModule &M) const override {
|
||||
|
@ -5486,7 +5486,7 @@ public:
|
|||
}
|
||||
|
||||
StringRef getARCRetainAutoreleasedReturnValueMarker() const override {
|
||||
return "mov\tr7, r7\t\t@ marker for objc_retainAutoreleaseReturnValue";
|
||||
return "mov\tr7, r7\t\t// marker for objc_retainAutoreleaseReturnValue";
|
||||
}
|
||||
|
||||
bool initDwarfEHRegSizeTable(CodeGen::CodeGenFunction &CGF,
|
||||
|
|
|
@ -13,7 +13,7 @@ id test0(void) {
|
|||
void test1(void) {
|
||||
extern id test1_helper(void);
|
||||
// CHECK: [[T0:%.*]] = call [[CC]]i8* @test1_helper()
|
||||
// CHECK-NEXT: call void asm sideeffect "mov
|
||||
// CHECK-NEXT: call void asm sideeffect "mov\09{{fp, fp|r7, r7}}\09\09// marker for objc_retainAutoreleaseReturnValue"
|
||||
// CHECK-NEXT: [[T1:%.*]] = call [[CC]]i8* @objc_retainAutoreleasedReturnValue(i8* [[T0]])
|
||||
// CHECK-NEXT: store i8* [[T1]],
|
||||
// CHECK-NEXT: call [[CC]]void @objc_storeStrong(
|
||||
|
|
Loading…
Reference in New Issue