dstenb (David Stenberg)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 10 2016, 1:55 AM (97 w, 1 d)

Recent Activity

Wed, Apr 18

Herald updated subscribers of D42848: Correct dwarf unwind information in function epilogue.
Wed, Apr 18, 7:39 AM

Mon, Apr 16

dstenb created D45686: [Tooling] Clean up tmp files when creating a fixed compilation database.
Mon, Apr 16, 7:17 AM
dstenb added a comment to D44568: Fix emission of phony dependency targets when adding extra deps.

Ping.

Mon, Apr 16, 4:54 AM

Wed, Apr 11

dstenb added a comment to D36827: Changed createTemporaryFile without FD to actually create a file..

Hi! We have encountered a regression where clang-tidy leaves behind temporary files after this change. I wrote a PR for that: https://bugs.llvm.org/show_bug.cgi?id=37091.

Wed, Apr 11, 10:04 AM

Wed, Mar 28

dstenb added a comment to D44774: [Driver] Allow use of -fsyntax-only together with -MJ.

Our legacy frontend does not support -MJ, so when using that frontend for code generation, we invoke clang with -MJ, and at the same use -fsyntax-only to get the improved diagnostics that clang provides. This is idiosyncratic and probably hacky, I know, but it works well enough to for example for getting access to defines and include flags from the compilation database, and being able to run clang-tidy. So (1) does not fit our use case, unfortunately.

Wed, Mar 28, 7:22 AM
dstenb added a comment to D44054: [DWARFv5] Emit file 0 to the line table..

With this change, we will emit the .file 0 directive even for -gdwarf-[234]. The directive results in an error when assembling with GAS. Should this be seen as a problem, or is it something that we accept?

Wed, Mar 28, 6:25 AM · debug-info

Mon, Mar 26

dstenb added a comment to D44774: [Driver] Allow use of -fsyntax-only together with -MJ.

Downstream we use -MJ in a bit of an idiosyncratic way, as we're in a transition period where we, for a subset of the code base, only use the clang frontend for diagnostics, and not for the code generation. However, if you don't think that using -fsyntax-only and -MJ makes sense in any upstream application, I'll drop from this change. I'm leaving the assertion as-is.

Mon, Mar 26, 7:22 AM

Mar 22 2018

dstenb created D44774: [Driver] Allow use of -fsyntax-only together with -MJ.
Mar 22 2018, 3:19 AM

Mar 16 2018

dstenb added a comment to D44568: Fix emission of phony dependency targets when adding extra deps.

A small caveat with this patch is that it does not fix the case where the input file as also added as an extra dependency with -fdepfile-entry; however, I reasoned that it shouldn't really be a problem in practice. I thought that it was a good trade-off ignoring that for slightly simpler code.

Mar 16 2018, 8:18 AM
dstenb created D44568: Fix emission of phony dependency targets when adding extra deps.
Mar 16 2018, 8:14 AM

Mar 2 2018

dstenb committed rL326573: Test commit: Remove an extraneous space. NFC.
Test commit: Remove an extraneous space. NFC
Mar 2 2018, 6:31 AM

Feb 1 2018

dstenb updated the diff for D42807: [GlobalOpt] Include padding in debug fragments.

Removed nonessential attributes.

Feb 1 2018, 10:55 AM · debug-info
dstenb created D42807: [GlobalOpt] Include padding in debug fragments.
Feb 1 2018, 10:39 AM · debug-info

Jan 26 2018

dstenb added inline comments to D42527: [DebugInfo] Fix fragment offset emission order for symbol locations.
Jan 26 2018, 4:36 AM · debug-info
dstenb added reviewers for D42527: [DebugInfo] Fix fragment offset emission order for symbol locations: aprantl, probinson, dblaikie.

Added reviewers.

Jan 26 2018, 1:03 AM · debug-info

Jan 25 2018

dstenb created D42527: [DebugInfo] Fix fragment offset emission order for symbol locations.
Jan 25 2018, 4:17 AM · debug-info

Jan 24 2018

dstenb created D42489: [GlobalOpt] Emit fragments using field offsets from struct layout.
Jan 24 2018, 8:46 AM

Oct 18 2017

dstenb added a comment to D33889: Mark intrinsics as norecurse.

Unless I'm mistaken, int_experimental_gc_statepoint should also be marked recursive?

Oct 18 2017, 8:54 AM

Sep 26 2017

dstenb updated the diff for D38172: [Debug info] Handle endianness when moving debug info for split integer values.

Simplified the test case according to dblaikie's comment (thanks!).

Sep 26 2017, 3:30 PM

Sep 25 2017

dstenb added a comment to D38172: [Debug info] Handle endianness when moving debug info for split integer values.

LGTM, but maybe wait for the other to have a look too before you land this.

Sep 25 2017, 5:03 AM

Sep 22 2017

dstenb created D38172: [Debug info] Handle endianness when moving debug info for split integer values.
Sep 22 2017, 4:24 AM

Aug 23 2017

dstenb added inline comments to D36865: [Reassociate] Do not drop debug location if replacement is missing.
Aug 23 2017, 1:07 AM
dstenb updated the diff for D36865: [Reassociate] Do not drop debug location if replacement is missing.

Changed so that the debug location is kept in the other cases.
Added CHECK statements (thanks for pointing that out!).

Aug 23 2017, 1:03 AM

Aug 21 2017

dstenb added a comment to D36865: [Reassociate] Do not drop debug location if replacement is missing.

I fully understand the rationale behind this patch and I think it is good, but why is it ok/desirable to drop the debug info in all other cases?

Aug 21 2017, 4:59 AM

Aug 18 2017

dstenb updated the summary of D36865: [Reassociate] Do not drop debug location if replacement is missing.
Aug 18 2017, 2:37 AM
dstenb created D36865: [Reassociate] Do not drop debug location if replacement is missing.
Aug 18 2017, 2:34 AM

Oct 13 2016

dstenb added reviewers for D25460: Fix memory issue in AttrBuilder::removeAttribute uses: mkuper, rnk.
Oct 13 2016, 1:47 AM

Oct 11 2016

dstenb retitled D25460: Fix memory issue in AttrBuilder::removeAttribute uses from to Fix memory issue in AttrBuilder::removeAttribute uses.
Oct 11 2016, 1:08 AM

Jun 20 2016

dstenb added inline comments to D21226: [SROA] Fix non-determinism caused by differing argument evaluation order .
Jun 20 2016, 4:35 AM
dstenb updated the diff for D21226: [SROA] Fix non-determinism caused by differing argument evaluation order .
Jun 20 2016, 12:49 AM
dstenb added a reviewer for D21226: [SROA] Fix non-determinism caused by differing argument evaluation order : patrik.h.hagglund.
Jun 20 2016, 12:40 AM

Jun 10 2016

dstenb retitled D21226: [SROA] Fix non-determinism caused by differing argument evaluation order from to [SROA] Fix non-determinism caused by differing argument evaluation order .
Jun 10 2016, 5:21 AM