https://github.com/DanielKristofKiss closed
https://github.com/llvm/llvm-project/pull/94271
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,13 @@
+// Check that -ffixed register handled for globals.
+// Regression test for #76426
+// RUN: %clang --target=aarch64-none-gnu -ffixed-x15 -### %s 2>&1 | FileCheck
%s
+// CHECK-NOT: fatal error: error in backend: Invalid register name "x15".
Danie
https://github.com/DanielKristofKiss updated
https://github.com/llvm/llvm-project/pull/94271
>From a8dabc508583ffb12982549d1bfb86cf6845c957 Mon Sep 17 00:00:00 2001
From: Daniel Kiss
Date: Sun, 21 Apr 2024 16:14:25 +0200
Subject: [PATCH] [AArch64] Add validation for Global Register Variable.
F
https://github.com/kawashima-fj edited
https://github.com/llvm/llvm-project/pull/94271
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,13 @@
+// Check that -ffixed register handled for globals.
+// Regression test for #76426
+// RUN: %clang --target=aarch64-none-gnu -ffixed-x15 -### %s 2>&1 | FileCheck
%s
+// CHECK-NOT: fatal error: error in backend: Invalid register name "x15".
kawas
https://github.com/kawashima-fj approved this pull request.
Thanks for the fix. I left a minor suggestion.
Some notes:
- This commit make Clang emit `error: size of register '???' does not match
variable size` error instead of `fatal error: error in backend: Invalid
register name "???"`, rega
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/94271
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kawashima-fj wrote:
@DanielKristofKiss Thanks for the fix. At first glance, LGTM. I'll take a
closer look in a few days.
I'm not so familiar with the code. If someone else can review, I'll leave it to
them.
https://github.com/llvm/llvm-project/pull/94271
___
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Daniel Kiss (DanielKristofKiss)
Changes
Fixes: #76426
---
Full diff: https://github.com/llvm/llvm-project/pull/94271.diff
2 Files Affected:
- (modified) clang/lib/Basic/Targets/AArch64.h (+12)
- (added) clang/test/Driver/aarch64-fixed-
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Daniel Kiss (DanielKristofKiss)
Changes
Fixes: #76426
---
Full diff: https://github.com/llvm/llvm-project/pull/94271.diff
2 Files Affected:
- (modified) clang/lib/Basic/Targets/AArch64.h (+12)
- (added) clang/test/Driver/aarch64
https://github.com/DanielKristofKiss created
https://github.com/llvm/llvm-project/pull/94271
Fixes: #76426
>From cda747d06c2d363c6c0c7f7d72825d119240ac5a Mon Sep 17 00:00:00 2001
From: Daniel Kiss
Date: Sun, 21 Apr 2024 16:14:25 +0200
Subject: [PATCH] [AArch64] Add validation for Global Regist
11 matches
Mail list logo