=?utf-8?q?Csan=C3=A1d_Hajd=C3=BA?= ,
=?utf-8?q?Csan=C3=A1d_Hajd=C3=BA?=
Message-ID:
In-Reply-To:
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/140554
___
cfe-commits mailing list
cfe-commits@lists.l
https://github.com/MaskRay commented:
Can you add a `clang/test/Misc/cc1as-*` test?
https://github.com/llvm/llvm-project/pull/159219
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/157321
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay wrote:
Can you add a test to `clang/test/Preprocessor/init.c`?
https://github.com/llvm/llvm-project/pull/153925
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4268,6 +4268,10 @@ def floop_interchange : Flag<["-"],
"floop-interchange">, Group,
HelpText<"Enable the loop interchange pass">, Visibility<[ClangOption,
CC1Option, FlangOption, FC1Option]>;
def fno_loop_interchange: Flag<["-"], "fno-loop-interchange">, Group,
HelpTe
@@ -6870,6 +6870,8 @@ void Clang::ConstructJob(Compilation &C, const JobAction
&JA,
options::OPT_fno_unroll_loops);
Args.AddLastArg(CmdArgs, options::OPT_floop_interchange,
options::OPT_fno_loop_interchange);
+ Args.AddLastArg(CmdArgs, op
MaskRay wrote:
You need to click "Re-request review" from folks who have requested changes.
https://github.com/llvm/llvm-project/pull/142686
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com
@@ -61,7 +62,14 @@ createLoongArchMCSubtargetInfo(const Triple &TT, StringRef
CPU, StringRef FS) {
static MCAsmInfo *createLoongArchMCAsmInfo(const MCRegisterInfo &MRI,
const Triple &TT,
cons
@@ -0,0 +1,36 @@
+//===- LoongArchWinCOFFStreamer.cpp -*- C++
-*-===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
MaskRay wrote:
I am concerned with auto -lstdc++fs as well. I think old GCC versions require
-lstdc++fs. If you are going to make changes in this area, especially for the
old versions [5.3, 9.2), please restrict them to AMD only.
https://github.com/llvm/llvm-project/pull/154826
___
https://github.com/MaskRay approved this pull request.
Please copy the original description to the description .
https://github.com/llvm/llvm-project/pull/154487
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
MaskRay wrote:
> LLVM Buildbot has detected a new failure on builder
> `llvm-clang-x86_64-sie-ubuntu-fast` running on `sie-linux-worker` while
> building `clang` at step 6 "test-build-unified-tree-check-all".
>
> Full details are available at:
> [lab.llvm.org/buildbot#/builders/144/builds/331
@@ -0,0 +1,66 @@
+// -*- C++ -*-
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-
@@ -0,0 +1,66 @@
+// -*- C++ -*-
MaskRay wrote:
Delete `// -*- C++ -*-`
https://github.com/llvm/llvm-project/pull/153565
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/li
@@ -397,6 +399,98 @@ void llvm::embedBufferInModule(Module &M, MemoryBufferRef
Buf,
appendToCompilerUsed(M, GV);
}
+void llvm::lowerIFuncsOnAIX(Module &M) {
MaskRay wrote:
Should not introduce new target-specific intrinsic uses in lib/Transform. I
think t
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/152942
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1044,9 +1044,10 @@
DEFINE_LIBUNWIND_FUNCTION(_ZN9libunwind18Registers_mips_o326jumptoEv)
lw$27, (4 * 27)($4)
lw$28, (4 * 28)($4)
lw$29, (4 * 29)($4)
- lw$30, (4 * 30)($4)
// load new pc into ra
lw$31, (4 * 32)($4)
+ //allow for load delay,
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/152942
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay commented:
Is this for PlayStation (MIPS 1)? I see that https://reviews.llvm.org/D122427
(2022) added some load delay slot support.
https://github.com/llvm/llvm-project/pull/152942
___
cfe-commits mailing list
cfe-commits@li
MaskRay wrote:
> `test/Rewrite` is for the Objective-C => MSVC-compatible C++ rewriter
> (`-rewrite-objc`), which is a different feature. That feature might _also_ be
> obsolete, but we haven't signed off on that yet AFAIK.
If cannot drop CLANG_ENABLE_OBJC_REWRITER code yet, defaulting to off
@@ -1671,6 +1671,11 @@ void CompilerInvocationBase::GenerateCodeGenArgs(const
CodeGenOptions &Opts,
else
GenerateArg(Consumer, OPT_fno_loop_interchange);
+ if (Opts.FuseLoops)
+GenerateArg(Consumer, OPT_fexperimental_loop_fusion);
+ else
MaskRay w
@@ -4268,6 +4268,10 @@ def floop_interchange : Flag<["-"],
"floop-interchange">, Group,
HelpText<"Enable the loop interchange pass">, Visibility<[ClangOption,
CC1Option, FlangOption, FC1Option]>;
def fno_loop_interchange: Flag<["-"], "fno-loop-interchange">, Group,
HelpTe
@@ -203,6 +203,19 @@
// RUN: | FileCheck -check-prefix=CHECK-RV64-GNU-RELAX %s
// CHECK-RV64-GNU-RELAX-NOT: "--no-relax"
+// Check that "-static -pie" is forwarded to linker when "-static-pie" is used
MaskRay wrote:
`/// Check` to make non-RUN-non-CHECK com
@@ -599,11 +599,18 @@ void baremetal::Linker::ConstructJob(Compilation &C,
const JobAction &JA,
const Driver &D = getToolChain().getDriver();
const llvm::Triple::ArchType Arch = TC.getArch();
const llvm::Triple &Triple = getToolChain().getEffectiveTriple();
+ const bool
Author: Fangrui Song
Date: 2025-08-05T10:02:53-07:00
New Revision: 913c5b4d1fffa00585edb6ba4f0915f29afa90cc
URL:
https://github.com/llvm/llvm-project/commit/913c5b4d1fffa00585edb6ba4f0915f29afa90cc
DIFF:
https://github.com/llvm/llvm-project/commit/913c5b4d1fffa00585edb6ba4f0915f29afa90cc.diff
MaskRay wrote:
I think not using SHF_EXCLUDE is an intention design. There are some
discussions on
https://discourse.llvm.org/t/end-to-end-fembed-bitcode-llvmbc-and-llvmcmd/56265
The concatenated bitcode might allow analysis like
https://github.com/travitch/whole-program-llvm (though it doesn
MaskRay wrote:
This is incorrect. Some use cases require the sections to be combined during
linking and adding ELF SHF_EXCLUDE would break the usage.
https://reviews.llvm.org/D86374
If you want these flags to be relocatable files only, specify `/DISCARD/` in a
linker script to discard them or
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/149988
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay approved this pull request.
LGTM! Thanks for not adding an extra driver option.
https://github.com/llvm/llvm-project/pull/149003
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/l
MaskRay wrote:
You might consider adding more reviewers to gather broader perspectives on this
feature. I'm neutral about it personally.
https://github.com/llvm/llvm-project/pull/149886
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://l
MaskRay wrote:
This looks quite good. I'm curious if others have any feedback.
https://github.com/llvm/llvm-project/pull/145056
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -450,6 +454,11 @@ class Driver {
return ClangExecutable.c_str();
}
+ StringRef getPreferredLinker() const { return PreferredLinker; }
+ void setPreferredLinker(std::string Value) {
MaskRay wrote:
If #149786 will use `setPreferredLinker`, this `setP
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/149784
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay wrote:
GlobPattern is also used in binary utilities' symbol/section name matching
where we want to be rigid. While I do not oppose to this change, I think it
should be opt-in if we add it.
https://github.com/llvm/llvm-project/pull/149886
___
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/149652
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay wrote:
Can you add a test to clang/test/Driver to show the behavior difference ?
https://github.com/llvm/llvm-project/pull/149681
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
MaskRay wrote:
Add a fclang test?
https://github.com/llvm/llvm-project/pull/149652
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -30,11 +30,16 @@ SanitizerSpecialCaseList::create(const
std::vector &Paths,
std::unique_ptr
SanitizerSpecialCaseList::createOrDie(const std::vector &Paths,
- llvm::vfs::FileSystem &VFS) {
+ llvm::vfs
MaskRay wrote:
> This is so that it's performed globally and not just for clang.
Can you elaborate what is "globally"?
https://github.com/llvm/llvm-project/pull/149652
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-
@@ -5895,6 +5895,11 @@ def o : JoinedOrSeparate<["-"], "o">,
Visibility<[ClangOption, CC1Option, CC1AsOption, FC1Option, FlangOption]>,
HelpText<"Write output to ">, MetaVarName<"">,
MarshallingInfoString>;
+def foutput_file_base : Joined<["-"], "foutput-file-base=">,
---
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/133661
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/147265
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2,42 +2,47 @@
/// Check DTLTO options are forwarded to the linker.
-// RUN: echo "--target=x86_64-linux-gnu \
-// RUN: -Xthinlto-distributor=distarg1 \
-// RUN: -Xthinlto-distributor=distarg2,distarg3 \
-// RUN: -fuse-ld=lld" > %t.rsp
-
/// Check that options are f
MaskRay wrote:
> I forgot about tomorrow's LLVM 21 branch so maybe I should merge this based
> on the support I've received so far?
Looks good!
> https://github.com/toppers what's the status of this in GCC? If it's in there
> I think we should land/backport this, otherwise after the branch is
MaskRay wrote:
This PR actually enables `-fasynchronous-unwind-tables` instead of
`-funwind-tables`.
`-fasynchronous-unwind-tables` and `-funwind-tables` control different boolean
values. When both are enabled, `-fasynchronous-unwind-tables` takes precedence.
https://github.com/llvm/llvm-pro
https://github.com/MaskRay approved this pull request.
Thanks! The reactions include a thumbs-up and a love. Setting my approval.
https://github.com/llvm/llvm-project/pull/148321
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llv
@@ -403,6 +403,11 @@
// MCPU-MARCH: "-nostdsysteminc" "-target-cpu" "sifive-e31" "-target-feature"
"+m" "-target-feature" "+c"
// MCPU-MARCH: "-target-abi" "ilp32"
+// march=unset erases previous march
MaskRay wrote:
`/// -march=unset erases previous -march`
@@ -0,0 +1,43 @@
+// REQUIRES: lld
+
+/// Check DTLTO options are forwarded to the linker.
+
+// RUN: echo "--target=x86_64-linux-gnu \
+// RUN: -Xthinlto-distributor=distarg1 \
+// RUN: -Xthinlto-distributor=distarg2,distarg3 \
+// RUN: -fuse-ld=lld" > %t.rsp
+
+/// Check t
@@ -0,0 +1,33 @@
+// RUN: %clang -flto=thin --target=x86_64-scei-ps4 -O2 -c %s -o %t.ps4.tu
-Xclang -fdebug-pass-manager 2>%t.ps4.tu.txt
MaskRay wrote:
unified-lto-pipeline.c is present and serves the purpose. For target-generic
codegen features, we usually tes
MaskRay wrote:
Thank you for your efforts to enhance sanitizers. However, the current
implementation aligns with the preferences of many, including GCC contributors
who introduced -fsanitize-trap in 2022. Gaining consensus to modify
-fsanitize-trap to issue warnings or errors seems unlikely, s
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/148321
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -403,6 +403,11 @@
// MCPU-MARCH: "-nostdsysteminc" "-target-cpu" "sifive-e31" "-target-feature"
"+m" "-target-feature" "+c"
// MCPU-MARCH: "-target-abi" "ilp32"
+// march=unset erases previous march
+// RUN: %clang --target=riscv32 -### -c %s 2>&1 -march=rv32imc -march=uns
https://github.com/MaskRay commented:
Thanks for implementing -mcpu=unset. This looks good. I'm glad to see
-mcpu=unset gaining support, especially after feeling like the lone voice in
the GCC thread: https://gcc.gnu.org/pipermail/gcc-patches/2025-June/685537.html
https://github.com/llvm/llvm-
https://github.com/MaskRay closed
https://github.com/llvm/llvm-project/pull/147997
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay wrote:
The current -fsanitize-trap behavior is intentional. See
https://discourse.llvm.org/t/clang-gsoc-2025-usability-improvements-for-trapping-undefined-behavior-sanitizer/84568/11
I believe Google actually depended on -fsanitize-trap=undefined not leading to
warnings/errors when -fs
@@ -12,7 +12,7 @@
target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
-target triple = "x86_64-unknown-linux-gnu"
+target triple = "x86_64-unknown--"
MaskRay wrote:
Yes, they can be omitted in the IR.
https://github.com/llvm/llvm-project/pull/12387
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/147589
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/146795
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
=?utf-8?q?Csanád_Hajdú?= ,
=?utf-8?q?Csanád_Hajdú?=
Message-ID:
In-Reply-To:
@@ -16,6 +16,8 @@
#if defined(_AIX)
.toc
+#elif defined(__aarch64__) && defined(__ELF__) &&
defined(_LIBUNWIND_EXECUTE_ONLY_CODE)
MaskRay wrote:
`defined(__ELF__)` seems unnece
https://github.com/MaskRay commented:
Driver change looks reasonable if samplepgo maintainers decide to proceed.
https://github.com/llvm/llvm-project/pull/146795
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
@@ -651,8 +651,19 @@ void AVR::Linker::ConstructJob(Compilation &C, const
JobAction &JA,
// This is almost always required because otherwise avr-ld
// will assume 'avr2' and warn about the program being larger
// than the bare minimum supports.
- if (Linker.find("avr-ld
@@ -263,11 +263,17 @@ bool AVRAsmPrinter::doFinalization(Module &M) {
auto *Section = cast(TLOF.SectionForGlobal(&GO, TM));
if (Section->getName().starts_with(".data"))
NeedsCopyData = true;
-else if (Section->getName().starts_with(".rodata") && SubTM->hasLPM(
@@ -263,11 +263,17 @@ bool AVRAsmPrinter::doFinalization(Module &M) {
auto *Section = cast(TLOF.SectionForGlobal(&GO, TM));
if (Section->getName().starts_with(".data"))
NeedsCopyData = true;
-else if (Section->getName().starts_with(".rodata") && SubTM->hasLPM(
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/146244
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/146244
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/146449
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay wrote:
> > Thanks for working on this! This will be the first time I'm not going to
> > oppose an effort to remove Native Client support 🎉 Although I am going to
> > ask you to wait a couple of months to land it, until we finally turn it off
> > for good and start deleting the support
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/145685
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/145685
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -349,7 +350,12 @@ class Triple {
/// triple fields unknown.
Triple() = default;
+ LLVM_ABI explicit Triple(std::string &&Str);
+ explicit Triple(StringRef Str) : Triple(Str.str()) {}
+ explicit Triple(const char *Str) : Triple(std::string(Str)) {}
+ explicit Triple(
MaskRay wrote:
Sorry, I haven't closely followed the AArch64 PAuth work. The Clang driver
options should come last in the patch stack, with other patches reviewed first.
For driver options, I don't know a bunch of `-fptrauth-*` is a good idea. We
had a similar scenario with -mbranches-within-3
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/139271
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,221 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
https://github.com/MaskRay approved this pull request.
LGTM. Thanks for removing this tech debt.
https://github.com/llvm/llvm-project/pull/144222
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cf
https://github.com/MaskRay commented:
LGTM!
https://github.com/llvm/llvm-project/pull/134621
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1635,8 +1656,13 @@ bool tools::addSanitizerRuntimes(const ToolChain &TC,
const ArgList &Args,
}
// If there is a static runtime with no dynamic list, force all the symbols
// to be dynamic to be sure we export sanitizer interface functions.
- if (AddExportDynamic)
-
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/143124
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -18,6 +18,53 @@
using namespace llvm;
using namespace AMDGPU;
+/// Find KV in array using binary search.
+static const BasicSubtargetSubTypeKV *
+Find(StringRef S, ArrayRef A) {
MaskRay wrote:
`find` (functionName in new code)
https://github.com/llvm/llvm
@@ -18,6 +18,53 @@
using namespace llvm;
using namespace AMDGPU;
+/// Find KV in array using binary search.
+static const BasicSubtargetSubTypeKV *
+Find(StringRef S, ArrayRef A) {
+ // Binary search the array
+ auto F = llvm::lower_bound(A, S);
+ // If not found then retur
@@ -0,0 +1,19 @@
+// RUN: rm -rf %t
+// RUN: mkdir -p %t
+
+// Create PCH without -ignore-pch.
+// RUN: %clang -x c++-header %S/../Modules/Inputs/codegen-flags/foo.h -###
2>&1 | FileCheck %s -check-prefix=CHECK-EMIT-PCH
MaskRay wrote:
`%S/../` references auxilia
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/143533
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/143268
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay wrote:
-mlong-calls is an old-fashioned compiler option. I think it was added before
linkers knew range extension thunks (aka stubs, veneers, etc).
Can you use -fno-plt instead? It works with both SelectionDAG and GlobalISel.
You will get GOT-generating code sequence that can be optimi
MaskRay wrote:
Should call this "option". Within LLVM, we use flag for options without a value.
https://github.com/llvm/llvm-project/pull/142901
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe
@@ -16,6 +16,7 @@
#include "llvm/ADT/StringMap.h"
#include "llvm/ADT/StringRef.h"
+#include "llvm/MC/MCSubtargetInfo.h"
MaskRay wrote:
@diggerlin This is blocking and should be addressed before landing.
https://github.com/llvm/llvm-project/pull/137670
__
https://github.com/MaskRay commented:
LGTM
https://github.com/llvm/llvm-project/pull/141562
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/142396
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay wrote:
> Thanks for working on this! This will be the first time I'm not going to
> oppose an effort to remove Native Client support 🎉 Although I am going to ask
> you to wait a couple of months to land it, until we finally turn it off for
> good and start deleting the support code fro
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/140556
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -614,3 +614,30 @@ nvfortran defines `-fast` as
- `-Mcache_align`: there is no equivalent flag in Flang or Clang.
- `-Mflushz`: flush-to-zero mode - when `-ffast-math` is specified, Flang will
link to `crtfastmath.o` to ensure denormal numbers are flushed to zero.
+
+
+#
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/140556
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -614,3 +614,30 @@ nvfortran defines `-fast` as
- `-Mcache_align`: there is no equivalent flag in Flang or Clang.
- `-Mflushz`: flush-to-zero mode - when `-ffast-math` is specified, Flang will
link to `crtfastmath.o` to ensure denormal numbers are flushed to zero.
+
+
+#
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/132680
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,12 @@
+! RUN: env FCC_OVERRIDE_OPTIONS="#+-Os +-Oz +-O +-O3 +-Oignore +a +b +c xb Xa
Omagic ^-### " %flang -target x86_64-unknown-linux-gnu %s -O2 b -O3 2>&1 |
FileCheck %s
MaskRay wrote:
`--target=x86_64-unknown-linux-gnu` instead of the long depre
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/140556
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay commented:
clang driver code looks good
https://github.com/llvm/llvm-project/pull/140556
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay wrote:
> Thank you for confirming.
>
> @MaskRay, do you have any comments regarding the handling of warning messages?
Sorry for the delayed response. Does this implement the last-option-win
behavior as suggested in the last few comments of
https://discourse.llvm.org/t/the-priority-of
@@ -2997,6 +2997,7 @@ static void RenderFloatingPointOptions(const ToolChain
&TC, const Driver &D,
LangOptions::ComplexRangeKind Range = LangOptions::ComplexRangeKind::CX_None;
std::string ComplexRangeStr = "";
std::string GccRangeComplexOption = "";
+ std::string LastC
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/141921
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -27,3 +27,10 @@
// RUN: not %clang --target=x86_64 -### -mhtm -lc %s 2>&1 \
// RUN: | FileCheck %s -check-prefix=UNSUP_OPT
// UNSUP_OPT: error: unsupported option
+
+
+// RUN: not %clang -c --target=aarch64-- -mfpu=crypto-neon-fp-armv8 %s 2>&1 | \
+// RUN: FileCheck %s --c
1 - 100 of 1904 matches
Mail list logo
