[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

https://github.com/MaskRay closed https://github.com/llvm/llvm-project/pull/85350 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

https://github.com/MaskRay updated https://github.com/llvm/llvm-project/pull/85350 >From 94997d2d43a4db288e81e7b4c61d4562645cc5d8 Mon Sep 17 00:00:00 2001 From: Fangrui Song Date: Thu, 14 Mar 2024 19:04:53 -0700 Subject: [PATCH 1/2] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=

[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

https://github.com/MaskRay updated https://github.com/llvm/llvm-project/pull/85350 >From 94997d2d43a4db288e81e7b4c61d4562645cc5d8 Mon Sep 17 00:00:00 2001 From: Fangrui Song Date: Thu, 14 Mar 2024 19:04:53 -0700 Subject: [PATCH 1/2] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=

[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

https://github.com/MaskRay edited https://github.com/llvm/llvm-project/pull/85350 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

https://github.com/MaskRay edited https://github.com/llvm/llvm-project/pull/85350 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

https://github.com/MaskRay updated https://github.com/llvm/llvm-project/pull/85350 >From 94997d2d43a4db288e81e7b4c61d4562645cc5d8 Mon Sep 17 00:00:00 2001 From: Fangrui Song Date: Thu, 14 Mar 2024 19:04:53 -0700 Subject: [PATCH] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20initia?=

[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

https://github.com/MaskRay edited https://github.com/llvm/llvm-project/pull/85350 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

@@ -321,9 +321,11 @@ void aarch64::getAArch64TargetFeatures(const Driver , } } - if (Arg *A = Args.getLastArg(options::OPT_mno_unaligned_access, - options::OPT_munaligned_access)) { -if

[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

@@ -321,9 +321,11 @@ void aarch64::getAArch64TargetFeatures(const Driver , } } - if (Arg *A = Args.getLastArg(options::OPT_mno_unaligned_access, - options::OPT_munaligned_access)) { -if

[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

https://github.com/SixWeining approved this pull request. LGTM from LoongArch side as GCC only have `-m[no-]strict-align`. BTW, should clang release notes be updated for this change? https://github.com/llvm/llvm-project/pull/85350 ___ cfe-commits

[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

@@ -321,9 +321,11 @@ void aarch64::getAArch64TargetFeatures(const Driver , } } - if (Arg *A = Args.getLastArg(options::OPT_mno_unaligned_access, - options::OPT_munaligned_access)) { -if

[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

@@ -321,9 +321,11 @@ void aarch64::getAArch64TargetFeatures(const Driver , } } - if (Arg *A = Args.getLastArg(options::OPT_mno_unaligned_access, - options::OPT_munaligned_access)) { -if

[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

https://github.com/wangpc-pp approved this pull request. LGTM. cc @asb @topperc Some context of RISCV target: https://github.com/riscv-non-isa/riscv-c-api-doc/pull/62 https://github.com/llvm/llvm-project/pull/85350 ___ cfe-commits mailing list

[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

@@ -321,9 +321,11 @@ void aarch64::getAArch64TargetFeatures(const Driver , } } - if (Arg *A = Args.getLastArg(options::OPT_mno_unaligned_access, - options::OPT_munaligned_access)) { -if

[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

llvmbot wrote: @llvm/pr-subscribers-backend-loongarch Author: Fangrui Song (MaskRay) Changes GCC ports only supports one of the options, with -mstrict-align preferred by newer ports. And they reject adding such aliases (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111555). We should not

[clang] [Driver] Don't alias -mstrict-align to -mno-unaligned-access (PR #85350)

https://github.com/MaskRay created https://github.com/llvm/llvm-project/pull/85350 GCC ports only supports one of the options, with -mstrict-align preferred by newer ports. And they reject adding such aliases (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111555). We should not support aliases,