https://github.com/hokein closed https://github.com/llvm/llvm-project/pull/80277
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/aaronpuchert approved this pull request.
Thanks, looks good to me!
https://github.com/llvm/llvm-project/pull/80277
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hokein wrote:
> This sounds like a good idea!
Done.
https://github.com/llvm/llvm-project/pull/80277
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/hokein updated
https://github.com/llvm/llvm-project/pull/80277
>From bb1a8f86e0b9ce3748af03625757d20900e053fc Mon Sep 17 00:00:00 2001
From: Haojian Wu
Date: Thu, 1 Feb 2024 12:41:29 +0100
Subject: [PATCH 1/2] Thread safety analysis: provide printSCFG definition.
I'm calling
https://github.com/hokein updated
https://github.com/llvm/llvm-project/pull/80277
>From bb1a8f86e0b9ce3748af03625757d20900e053fc Mon Sep 17 00:00:00 2001
From: Haojian Wu
Date: Thu, 1 Feb 2024 12:41:29 +0100
Subject: [PATCH 1/2] Thread safety analysis: provide printSCFG definition.
I'm calling
aaronpuchert wrote:
This sounds like a good idea!
https://github.com/llvm/llvm-project/pull/80277
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hokein wrote:
> It might have been commented out so that it doesn't take up space in the
> compiled binary.
>
> I'd like seeing it compiled, just to make sure it doesn't break. But I'd also
> like if it doesn't appear in the final binary. Perhaps we can change
> visibility so that `--gc-secti
aaronpuchert wrote:
It might have been commented out so that it doesn't take up space in the
compiled binary.
I'd like seeing it compiled, just to make sure it doesn't break. But I'd also
like if it doesn't appear in the final binary. Perhaps we can change visibility
so that `--gc-sections` o
llvmbot wrote:
@llvm/pr-subscribers-clang-analysis
Author: Haojian Wu (hokein)
Changes
I called this function when investigating the issue
(https://github.com/llvm/llvm-project/issues/78131), and I was surprised to see
the definition is commented out.
I think it makes sense to provide t
https://github.com/hokein created
https://github.com/llvm/llvm-project/pull/80277
I called this function when investigating the issue
(https://github.com/llvm/llvm-project/issues/78131), and I was surprised to see
the definition is commented out.
I think it makes sense to provide the definiti
10 matches
Mail list logo