huangjd wrote:
Ok lets revert this first, and I will work on the dbg.value implementation
https://github.com/llvm/llvm-project/pull/94100
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
SLTozer wrote:
Yes, to my knowledge Swift emits dbg.values out of its front end, so there's
probably no reason (on the LLVM side) that clang couldn't do so as well. There
might be good reasons internal to clang that it doesn't use debug values atm,
and it's not inconceivable that we'd hit
jmorse wrote:
dblaikie wrote:
> I hope that isn't the intent, I thought the intent was just [...]
Ah, by which I mean "it's not an accident",
> By just specifying -g Even without -fdebug-info-for-profiling it is going to
> introduce debug variables as a sequence of alloca-store-load too, so
huangjd wrote:
By just specifying `-g` Even without `-fdebug-info-for-profiling` it is going
to introduce debug variables as a sequence of alloca-store-load too, so is it a
requirement to guarantee `-O0` output stays identical with or without debug
info?
Emitting a dbg.value was what I
dwblaikie wrote:
> G'day -- we've got some tests for -O0 that, with this patch applied, generate
> different code with-and-without the `-g` flag, if
> `-fdebug-info-for-profiling` is enabled. Example godbolt:
> https://godbolt.org/z/qooo5Eah1 .
>
> It seems the intention of this patch is
jmorse wrote:
G'day -- we've got some tests for -O0 that, with this patch applied, generate
different code with-and-without the `-g` flag, if `-fdebug-info-for-profiling`
is enabled. Example godbolt: https://godbolt.org/z/qooo5Eah1 .
It seems the intention of this patch is generating
https://github.com/huangjd closed
https://github.com/llvm/llvm-project/pull/94100
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
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 9261ab708e37c2d6499ac063045f816d25a5919c
4841c8752abab8c5e05a97d8826cdca376f4ab0c --
https://github.com/dwblaikie approved this pull request.
Looks worth a shot. Please commit this at a time where you can respond to bot
fail-mail for a while after the commit.
And ensure the commit message has most/all of the details of the original
commit, plus the original commit and revert
https://github.com/huangjd created
https://github.com/llvm/llvm-project/pull/94100
This is another attempt to land #81545, which was reverted.
Fixed test case so that it is consistent on all platforms.
>From f2c82758e1cba7773e41d941d2812c829c339675 Mon Sep 17 00:00:00 2001
From: William
10 matches
Mail list logo
