https://github.com/MaskRay requested changes to this pull request.
.
https://github.com/llvm/llvm-project/pull/113152
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-co
@@ -543,12 +543,59 @@ static void handlePAuthABI(const Driver &D, const ArgList
&DriverArgs,
CC1Args.push_back("-faarch64-jump-table-hardening");
}
+static void addPointerAuthFlags(const llvm::opt::ArgList &DriverArgs,
MaskRay wrote:
There is a lot of du
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/113152
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -59,4 +97,39 @@
// RUN: | FileCheck -check-prefix=CXX-ARM-BAREMETAL-NOSYSROOT-LIBCXX %s
// CXX-ARM-BAREMETAL-NOSYSROOT-LIBCXX: "-internal-isystem"
"{{.*}}/Inputs/basic_arm_gcc_tree/lib/gcc/armv6m-none-eabi/8.2.1/../../../../armv6m-none-eabi/include/c++/v1"
-// CXX-ARM-BA
@@ -14,12 +15,17 @@ __attribute__((noinline)) void init(S *s) {
__asm__ __volatile__("" : : "r"(s) : "memory");
}
-__attribute__((noinline, no_sanitize("memory"))) int test(char i) {
+__attribute__((noinline, no_sanitize("memory", "address", "hwaddress"))) int
+test(char i)
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/120515
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
MaskRay wrote:
Ping:)
https://github.com/llvm/llvm-project/pull/119835
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/125151
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay approved this pull request.
We try to support very few -Wa,... options for the integrated assembler, since
as you can see, the number of lines increases quite a lot. if this option is so
frequently used, i think this is ok.
https://github.com/llvm/llvm-project/pull/1
@@ -2842,6 +2868,12 @@ static void
CollectArgsForIntegratedAssembler(Compilation &C,
CmdArgs.push_back("-target-feature");
CmdArgs.push_back(MipsTargetFeature);
}
+ if (!SparcTargetFeatures.empty()) {
MaskRay wrote:
`if` can be dropped
https://git
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/126446
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -460,6 +460,8 @@ Changes to the LLVM tools
`--localize-symbol`, `--localize-symbols`,
`--skip-symbol`, `--skip-symbols`.
+* llvm-objcopy now prints the correct file path in the error message when the
output file specified by --dump-section cannot be opened.
---
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/126607
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -0,0 +1,73 @@
+; Tests that we store the type identifiers in .callgraph section of the binary.
+
+; RUN: llc --call-graph-section -filetype=obj -o - < %s | \
+; RUN: llvm-readelf -x .callgraph - | FileCheck %s
+
+target triple = "x86_64-unknown-linux-gnu"
MaskR
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/125334
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -15,50 +16,85 @@
// RUN: -fno-ptrauth-indirect-gotos -fptrauth-indirect-gotos \
// RUN: -fno-ptrauth-init-fini -fptrauth-init-fini \
// RUN: -fno-ptrauth-init-fini-address-discrimination
-fptrauth-init-fini-address-discrimination \
+// RUN: -fno-ptrauth-elf-got -fpt
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/113152
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/119835
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
MaskRay wrote:
Updated the description. This consolidates global variables.
https://github.com/llvm/llvm-project/pull/119835
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-b
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/119835
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/119835
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay created
https://github.com/llvm/llvm-project/pull/119835
None
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -78,6 +78,79 @@ _start:
adrp x1, :got_auth:zed
add x1, x1, :got_auth_lo12:zed
+#--- ok-tiny.s
+
+# RUN: llvm-mc -filetype=obj -triple=aarch64-none-linux ok-tiny.s -o ok-tiny.o
MaskRay wrote:
`none` is also an OS component and is misleading when used t
@@ -78,6 +78,79 @@ _start:
adrp x1, :got_auth:zed
add x1, x1, :got_auth_lo12:zed
+#--- ok-tiny.s
+
+# RUN: llvm-mc -filetype=obj -triple=aarch64-none-linux ok-tiny.s -o ok-tiny.o
+
+# RUN: ld.lld ok-tiny.o a.so -pie -o external-tiny
+# RUN: llvm-readelf -r -S -x .got exte
@@ -1150,6 +1152,58 @@ void LoongArch::tlsdescToLe(uint8_t *loc, const
Relocation &rel,
}
}
+// Try GOT indirection to PC relative optimization when relaxation is enabled.
+// From:
+// * pcalau12i $a0, %got_pc_hi20(sym_got)
+// * ld.w/d$a0, $a0, %got_pc_lo12(sym_got)
@@ -1150,6 +1152,58 @@ void LoongArch::tlsdescToLe(uint8_t *loc, const
Relocation &rel,
}
}
+// Try GOT indirection to PC relative optimization when relaxation is enabled.
+// From:
+// * pcalau12i $a0, %got_pc_hi20(sym_got)
+// * ld.w/d$a0, $a0, %got_pc_lo12(sym_got)
MaskRay wrote:
Can you drop the trailing `.` in the subject?
https://github.com/llvm/llvm-project/pull/123743
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/119835
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/119835
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/119835
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -48,7 +48,7 @@ ConfigWrapper config;
Ctx ctx;
MaskRay wrote:
Removed `ConfigWrapper config`. Yes, that was intended to be a follow-up, but
it seems easy to fold it into this PR.
https://github.com/llvm/llvm-project/pull/119835
__
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/119835
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -1248,6 +1248,8 @@ Sanitizers
by the compiler (for example,
``-fno-sanitize-merge=bool,enum,array-bounds,local-bounds``).
+- ``-fsanitize=vptr`` is not included into ``-fsanitize=undefined``.
MaskRay wrote:
no longer part of
https://github.com/llvm/ll
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/121115
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/123280
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -1750,7 +1750,9 @@ void
MappingTraits>::mapping(
std::string MappingTraits>::validate(
IO &io, std::unique_ptr &C) {
if (const auto *F = dyn_cast(C.get())) {
-if (F->Pattern && F->Pattern->binary_size() != 0 && !F->Size)
+// Can't check the `Size`, as it's req
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/121410
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -0,0 +1,115 @@
+# REQUIRES: loongarch
+
+# RUN: llvm-mc --filetype=obj --triple=loongarch32 -mattr=+relax --defsym
ELF32=1 %s -o %t.32.o
+# RUN: llvm-mc --filetype=obj --triple=loongarch64 -mattr=+relax %s -o %t.64.o
+
+# RUN: ld.lld %t.32.o -o %t.32
+# RUN: llvm-objdump -d --
https://github.com/MaskRay commented:
@xen0n May I ask you to double check the code? It seems good and appears to be
LGTMed by a colleague of the author.
https://github.com/llvm/llvm-project/pull/123600
___
llvm-branch-commits mailing list
llvm-branch
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/123600
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/132569
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -1,4 +1,237 @@
-! RUN: llvm-mc %s -triple=sparcv9 -mcpu=niagara -show-encoding | FileCheck %s
+! RUN: not llvm-mc %s -triple=sparcv9 -show-encoding 2>&1 | FileCheck %s
--check-prefixes=NO-VIS
MaskRay wrote:
might want `--implicit-check-not=error:` to ensure t
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/132569
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
MaskRay wrote:
> The linker job in BareMetal toolchain object will be used by gnuld and lld
> both.
GNU ld instead of gnuld.
https://github.com/llvm/llvm-project/pull/132806
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
http
@@ -2233,8 +2235,17 @@ ParseStatus
RISCVAsmParser::parseOperandWithSpecifier(OperandVector &Operands) {
SMLoc S = getLoc();
SMLoc E;
- if (!parseOptionalToken(AsmToken::Percent) ||
- getLexer().getKind() != AsmToken::Identifier)
+ if (!parseOptionalToken(AsmToken::
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/133348
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -18,6 +18,6 @@
.globl _start
_start:
.data
- .word foo@PLT - .
- .word foo@PLT - . + 1
- .word foo@PLT - . - 1
+ .word %plt(foo - .)
MaskRay wrote:
Implemented `%pltpcrel`, while a bit complex, I am happy with the result. (As
in `RISCVMCExpr::evaluateA
MaskRay wrote:
LGTM.
Note: test/CodeGen is probably not the best place for such tests.
`test/MC/CSKY/relocation-specifier.s` contains a nice example and tests
multiple relocations at the same time.
https://github.com/llvm/llvm-project/pull/133027
__
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/135123
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay created
https://github.com/llvm/llvm-project/pull/134879
now that llc supports `-M no-aliases` (along with llvm-mc and llvm-objdump).
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.ll
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/134879
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/133591
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -169,6 +169,91 @@ enum SubsectionKind : uint8_t {
SK_PPA1 = 2,
SK_PPA2 = 4,
};
+
+// The standard System/390 convention is to name the high-order (leftmost) bit
+// in a byte as bit zero. The Flags type helps to set bits in byte according
+// to this numeration order.
+c
@@ -0,0 +1,73 @@
+; RUN: llc <%s --mtriple s390x-ibm-zos --filetype=obj -o - | \
+; RUN: od -Ax -tx1 -v | FileCheck --ignore-case %s
+; REQUIRES: systemz-registered-target
MaskRay wrote:
unneeded thanks to
```
% cat llvm/test/MC/SystemZ/lit.local.cfg
if not "Sy
@@ -169,6 +169,91 @@ enum SubsectionKind : uint8_t {
SK_PPA1 = 2,
SK_PPA2 = 4,
};
+
+// The standard System/390 convention is to name the high-order (leftmost) bit
+// in a byte as bit zero. The Flags type helps to set bits in byte according
+// to this numeration order.
+c
https://github.com/MaskRay commented:
LGTM. As I'm not familiar with z/OS, my review focused on its compatibility
with the current MC infrastructure. (My internet access will be limited between
April 20th and May 4th, which may cause delays in my response time.)
Thanks for reimplementing the g
@@ -0,0 +1,236 @@
+## Check parsing of a .llvm_jump_table_info section
MaskRay wrote:
(Can use the llvm/utils/update_test_body.py format to make re-generation easier
:) )
https://github.com/llvm/llvm-project/pull/132114
_
MaskRay wrote:
> I had pushed #135324 a couple of days ago to fix the same issue. I'm happy to
> land either patch.
Thanks! I'll land this one (wanted to make it clear `@plt` is a legacy thing
and test `call foo@3`). Let me think about the AsmBackend change.
https://github.com/llvm/llvm-proje
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/135509
>From 3579f7d24a771f4c02e796a2ac9b91024b3c6ce9 Mon Sep 17 00:00:00 2001
From: Fangrui Song
Date: Sat, 12 Apr 2025 16:27:14 -0700
Subject: [PATCH] .
Created using spr 1.3.5-bogner
---
llvm/test/MC/RISCV/functio
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/135509
>From 3579f7d24a771f4c02e796a2ac9b91024b3c6ce9 Mon Sep 17 00:00:00 2001
From: Fangrui Song
Date: Sat, 12 Apr 2025 16:27:14 -0700
Subject: [PATCH] .
Created using spr 1.3.5-bogner
---
llvm/test/MC/RISCV/functio
https://github.com/MaskRay created
https://github.com/llvm/llvm-project/pull/135509
... instead of silently parsing and ignoring it without leaving an error
message.
While here, remove an unreachable `@plt`.
___
llvm-branch-commits mailing list
llv
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/135509
>From 3579f7d24a771f4c02e796a2ac9b91024b3c6ce9 Mon Sep 17 00:00:00 2001
From: Fangrui Song
Date: Sat, 12 Apr 2025 16:27:14 -0700
Subject: [PATCH] .
Created using spr 1.3.5-bogner
---
llvm/test/MC/RISCV/functio
@@ -1,5 +1,12 @@
// UNSUPPORTED: system-windows
+// Test interaction with -fuse-ld=lld
MaskRay wrote:
For `-fuse-ld=lld` , testing `ld.lld` is probably not a good use of an extra
RUN line. Could just modify one of the tests below from `-fuse-ld=` to
`-fuse-l
@@ -1,5 +1,12 @@
// UNSUPPORTED: system-windows
+// Test interaction with -fuse-ld=lld
MaskRay wrote:
You need to ensure that there is an `ld.lld` file with `-B%S/Inputs/lld`
https://github.com/llvm/llvm-project/pull/121830
___
MaskRay wrote:
> Hey Amir,
>
> Thanks for the PR. Unfortunately, it is still failing. The trick below
> doesn't seem to work on my buildbot machine:
>
> > Link against a DSO to ensure PLT entries.
>
> So doing:
>
> ```shell
> nm --synthetic callcont-fallthru.s.tmp
> ```
>
> won't list a `pu
@@ -599,8 +600,18 @@ class MCContext {
unsigned Flags,
unsigned EntrySize);
- MCSectionGOFF *getGOFFSection(StringRef Section, SectionKind Kind,
-
@@ -0,0 +1,112 @@
+; RUN: llc <%s --mtriple s390x-ibm-zos --filetype=obj -o - | \
MaskRay wrote:
`llc < %s` does not need `-o -`. .ll tests in llvm/test/MC is probably not a
good convention we'd recommend. Move to llvm/test/CodeGen/SystemZ, perhaps
under a dire
MaskRay wrote:
3
δΉ‹
https://github.com/llvm/llvm-project/pull/133799
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/135867
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
MaskRay wrote:
> The alternative fix, which I think I'm now leaning towards, would be to
> change how the branch-to-branch optimization handles relocations to
> STT_SECTION symbols. A relocation pointing to the STT_SECTION for .text with
> addend 1 would be treated as a branch to .text+5 and i
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/126848
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
MaskRay wrote:
> We're facing it again, with LLVM20 this time
... which indicates that the distro and boost should really fix the problem and
not rely on the clang driver hacks.
https://github.com/llvm/llvm-project/pull/102039
___
llvm-branch-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/129251
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -331,6 +333,32 @@ BareMetal::OrderedMultilibs
BareMetal::getOrderedMultilibs() const {
return llvm::reverse(Default);
}
+ToolChain::CXXStdlibType BareMetal::GetDefaultCXXStdlibType() const {
+ if (getTriple().isRISCV() && GCCInstallation.isValid())
+return ToolChain
@@ -830,6 +832,37 @@ static void relaxPCHi20Lo12(Ctx &ctx, const InputSection
&sec, size_t i,
remove = 4;
}
+// Relax code sequence.
+// From:
+// pcaddu18i $ra, %call36(foo)
+// jirl $ra, $ra, 0
+// To:
+// b/bl foo
+static void relaxCall36(Ctx &ctx, const InputSecti
@@ -830,6 +832,37 @@ static void relaxPCHi20Lo12(Ctx &ctx, const InputSection
&sec, size_t i,
remove = 4;
}
+// Relax code sequence.
+// From:
+// pcaddu18i $ra, %call36(foo)
+// jirl $ra, $ra, 0
+// To:
+// b/bl foo
+static void relaxCall36(Ctx &ctx, const InputSecti
@@ -0,0 +1,61 @@
+# REQUIRES: loongarch
MaskRay wrote:
omit `-2` from the name
https://github.com/llvm/llvm-project/pull/123576
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm
@@ -0,0 +1,135 @@
+# REQUIRES: loongarch
MaskRay wrote:
Needs a file-level comment explaining the purpose. See riscv-relax-call.s
https://github.com/llvm/llvm-project/pull/123576
___
llvm-branch-commits mailing list
ll
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/123702
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
MaskRay wrote:
A lot of utilities using LLVMOption, including clang and lld, are able to
report multiple unknown option errors or other parsing errors before exiting.
What the motivation behind this change?
https://github.com/llvm/llvm-project/pull/128447
__
MaskRay wrote:
The default value of `OutputFormat` is `berkeley`, so the assignment seems
redundant.
I see that `Radix` does need a change. Perhaps initialize the global variable
instead.
https://github.com/llvm/llvm-project/pull/128447
___
llvm-bran
@@ -19,7 +19,7 @@
; CHECK:.quad L#PPA2-CELQSTRT * A(PPA2-CELQSTRT)
; CHECK: L#PPA1_void_test_0:
; CHECK:.long L#PPA2-L#PPA1_void_test_0 * Offset to PPA2
-; CHECK:.section"B_IDRL"
+; CHECK:.section".idrl"
MaskRay
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/132569
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/132569
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/132569
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -18,6 +18,6 @@
.globl _start
_start:
.data
- .word foo@PLT - .
- .word foo@PLT - . + 1
- .word foo@PLT - . - 1
+ .word %plt(foo - .)
MaskRay wrote:
The core principle is that relocation specifiers must operate on the full
expression, not just a part of
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/132569
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -1194,18 +1194,34 @@ const MCExpr
*TargetLoweringObjectFileELF::lowerRelativeReference(
MCSymbolRefExpr::create(TM.getSymbol(RHS), getContext()), getContext());
MaskRay wrote:
`lowerRelativeReference` uses MCSymbolRefExpr. It seems legacy (probably sho
@@ -18,6 +18,6 @@
.globl _start
_start:
.data
- .word foo@PLT - .
- .word foo@PLT - . + 1
- .word foo@PLT - . - 1
+ .word %plt(foo - .)
MaskRay wrote:
It's challenging to use an inherent PC-relative specifier (e.g. `%pltpcrel`;
which I actually thought ab
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/132569
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay created
https://github.com/llvm/llvm-project/pull/132569
clang -fexperimental-relative-c++-abi-vtables might generate @plt and
@gotpcrel specifiers in data directives. The syntax is not used in
humand-written assembly code, and is not supported by GNU assembler.
Note:
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/132569
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -18,6 +18,6 @@
.globl _start
_start:
.data
- .word foo@PLT - .
- .word foo@PLT - . + 1
- .word foo@PLT - . - 1
+ .word %plt(foo - .)
MaskRay wrote:
The IR doesn't model the current location (DOT). Instead, It computes
`SymA-SymB+offset`.
`SymB` might n
@@ -2289,25 +2314,6 @@ bool
TargetLoweringObjectFileWasm::shouldPutJumpTableInFunctionSection(
return false;
}
-const MCExpr *TargetLoweringObjectFileWasm::lowerRelativeReference(
MaskRay wrote:
I'll pre-commit this. wasm and xcoff did cargo culting and ad
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/132569
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/132569
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/132806
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/132569
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/132569
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -169,6 +169,91 @@ enum SubsectionKind : uint8_t {
SK_PPA1 = 2,
SK_PPA2 = 4,
};
+
+// The standard System/390 convention is to name the high-order (leftmost) bit
+// in a byte as bit zero. The Flags type helps to set bits in byte according
+// to this numeration order.
+c
601 - 700 of 731 matches
Mail list logo
