labrinea wrote:
Update: The ACLE spec is now explicit about this (see
https://github.com/ARM-software/acle/pull/310), so we can safely implement the
change in the compiler. Implementation-wise I have moved the static cast to
`llvm::Function` inside the `createFunction()` lambda.
https://githu
https://github.com/labrinea edited
https://github.com/llvm/llvm-project/pull/85454
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/labrinea updated
https://github.com/llvm/llvm-project/pull/85454
>From 60a6fcd7560ec434a8369240f7a9cd13cf035c42 Mon Sep 17 00:00:00 2001
From: Alexandros Lamprineas
Date: Fri, 15 Mar 2024 19:25:16 +
Subject: [PATCH] [FMV] Allow multi versioning without default declaration
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff 8c4546f350fbce938d8fbc85d9e353d011f3f673
ec2af3b7f5936cc452251cd61a396329457038d3 --
labrinea wrote:
Hmm, I am now having second thoughts about this change. All is well without a
default declaration as long as we don't have a caller in the TU. In that case
(if there is a caller) a resolver is needed, but without a default declaration
we can't create a complete resolver (the re
labrinea wrote:
I think I messed up the review, this is based on #84405 so it shows unrelated
changes. Ignore for now. I'll rebase
https://github.com/llvm/llvm-project/pull/85454
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.ll
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Alexandros Lamprineas (labrinea)
Changes
This was a limitation which has now been lifted upon request.
Please read the thread below for more details:
https://github.com/llvm/llvm-project/pull/84405#discussion_r1525583647
B
https://github.com/labrinea created
https://github.com/llvm/llvm-project/pull/85454
This was a limitation which has now been lifted upon request.
Please read the thread below for more details:
https://github.com/llvm/llvm-project/pull/84405#discussion_r1525583647
Basically it allows to