Page MenuHomePhabricator
Feed Advanced Search

Mon, Jun 15

saugustine abandoned D81683: llc doesn't recognize "-debug" without asserts enabled..
Mon, Jun 15, 9:46 AM · Restricted Project

Fri, Jun 12

saugustine committed rGe64059828f19: Revert "[analyzer][NFC] Don't allow dependency checkers to emit diagnostics" (authored by saugustine).
Revert "[analyzer][NFC] Don't allow dependency checkers to emit diagnostics"
Fri, Jun 12, 12:36 PM
saugustine added a reverting change for rG33fb9cbe211d: [analyzer][NFC] Don't allow dependency checkers to emit diagnostics: rGe64059828f19: Revert "[analyzer][NFC] Don't allow dependency checkers to emit diagnostics".
Fri, Jun 12, 12:36 PM
saugustine closed D81752: Revert "[analyzer][NFC] Don't allow dependency checkers to emit diagnostics".
Fri, Jun 12, 12:35 PM · Restricted Project
saugustine added a comment to rG33fb9cbe211d: [analyzer][NFC] Don't allow dependency checkers to emit diagnostics.

Thanks for the quick response and easy deal. Looking forward to this relanding--it's a good change for sure.

Fri, Jun 12, 12:35 PM
saugustine added a reverting change for rG33fb9cbe211d: [analyzer][NFC] Don't allow dependency checkers to emit diagnostics: D81752: Revert "[analyzer][NFC] Don't allow dependency checkers to emit diagnostics".
Fri, Jun 12, 11:30 AM
saugustine created D81752: Revert "[analyzer][NFC] Don't allow dependency checkers to emit diagnostics".
Fri, Jun 12, 11:28 AM · Restricted Project
saugustine added inline comments to D78126: [analyzer][NFC] Don't allow dependency checkers to emit diagnostics.
Fri, Jun 12, 10:52 AM · Restricted Project

Thu, Jun 11

saugustine created D81683: llc doesn't recognize "-debug" without asserts enabled..
Thu, Jun 11, 12:41 PM · Restricted Project

Tue, Jun 9

saugustine committed rGccd192204da4: Fix variables used only in asserts. (authored by saugustine).
Fix variables used only in asserts.
Tue, Jun 9, 5:43 PM
saugustine closed D81522: Fix variables used only in asserts..
Tue, Jun 9, 5:43 PM · Restricted Project
saugustine updated the summary of D81522: Fix variables used only in asserts..
Tue, Jun 9, 5:42 PM · Restricted Project
saugustine added a comment to D80286: [Analyzer] Allow creation of stack frame for functions without definition.

Unfortunately, after this change there are several variables only used in asserts, which creates build failures when assertions are disabled.

Tue, Jun 9, 5:42 PM · Restricted Project
saugustine created D81522: Fix variables used only in asserts..
Tue, Jun 9, 5:42 PM · Restricted Project

Mon, Jun 8

saugustine committed rGf07b3d41e739: Add #includes so that ROCm.h is compilable stand-alone. (authored by saugustine).
Add #includes so that ROCm.h is compilable stand-alone.
Mon, Jun 8, 2:26 PM
saugustine closed D81432: Add #includes so that ROCm.h is compilable stand-alone..
Mon, Jun 8, 2:26 PM · Restricted Project
saugustine added a comment to D81432: Add #includes so that ROCm.h is compilable stand-alone..

For post-commit review.

Mon, Jun 8, 2:25 PM · Restricted Project
saugustine created D81432: Add #includes so that ROCm.h is compilable stand-alone..
Mon, Jun 8, 2:25 PM · Restricted Project
saugustine added reviewers for D81432: Add #includes so that ROCm.h is compilable stand-alone.: jyknight, echristo.
Mon, Jun 8, 2:25 PM · Restricted Project

Jun 1 2020

saugustine committed rGf027cfa37e67: For --relativenames, ignore directory 0, which is the comp_dir. (authored by saugustine).
For --relativenames, ignore directory 0, which is the comp_dir.
Jun 1 2020, 1:34 PM
saugustine closed D80168: For --relativenames, handle dwarf absolute include directories similarly to compilation directories..
Jun 1 2020, 1:33 PM · Restricted Project

May 29 2020

saugustine added a comment to D80168: For --relativenames, handle dwarf absolute include directories similarly to compilation directories..

I have added the necessary debug info to the .s file--it is a fairly minimal set. I have also removed the checked in binary.

May 29 2020, 2:14 PM · Restricted Project
saugustine updated the diff for D80168: For --relativenames, handle dwarf absolute include directories similarly to compilation directories..

Address upstream concerns about a checked-in binary for the test case.

May 29 2020, 2:14 PM · Restricted Project

May 28 2020

saugustine added a comment to D80168: For --relativenames, handle dwarf absolute include directories similarly to compilation directories..

Yeah, I'm kind of inclined to agree with @aprantl here - when the client asks for relative names I could see how that'd be reasonable to say "don't add the compilation dir" but I'm not sure what it means for other paths/doesn't necessarily seem reasonable for it to mean "give me back the unqualified name" even if it's relative to some other/unknown directory.

How would the client then know where to find this name relative to?

May 28 2020, 10:59 AM · Restricted Project
saugustine updated the diff for D80168: For --relativenames, handle dwarf absolute include directories similarly to compilation directories..

Update for upstream comments.

May 28 2020, 10:23 AM · Restricted Project

May 26 2020

saugustine added a comment to D79924: [gdb] Decay type before passing into printer.

This is puzzling to me, as we use this internally at Google all over the place, and it seems to work just fine. So I'm surprised we haven't noticed anything.

May 26 2020, 10:16 AM

May 21 2020

saugustine added inline comments to D80168: For --relativenames, handle dwarf absolute include directories similarly to compilation directories..
May 21 2020, 10:15 AM · Restricted Project
saugustine updated the diff for D80168: For --relativenames, handle dwarf absolute include directories similarly to compilation directories..

Address comments

May 21 2020, 10:15 AM · Restricted Project

May 19 2020

saugustine added a comment to D80168: For --relativenames, handle dwarf absolute include directories similarly to compilation directories..

Ping on this?

May 19 2020, 6:43 PM · Restricted Project

May 18 2020

saugustine created D80168: For --relativenames, handle dwarf absolute include directories similarly to compilation directories..
May 18 2020, 4:17 PM · Restricted Project

Apr 28 2020

saugustine committed rG3421d1ede4c1: Make getCompileUnitForAddress public. (authored by saugustine).
Make getCompileUnitForAddress public.
Apr 28 2020, 5:18 PM
saugustine closed D79048: Make getCompileUnitForAddress public..
Apr 28 2020, 5:18 PM · Restricted Project
saugustine created D79048: Make getCompileUnitForAddress public..
Apr 28 2020, 3:40 PM · Restricted Project
saugustine added reviewers for D79048: Make getCompileUnitForAddress public.: dblaikie, echristo.
Apr 28 2020, 3:40 PM · Restricted Project

Apr 23 2020

saugustine added inline comments to D70416: [Driver] Make -static-libgcc imply static libunwind.
Apr 23 2020, 4:52 PM · Restricted Project

Apr 17 2020

saugustine committed rGa4b88c044980: Revert "Implement _ExtInt as an extended int type specifier." (authored by saugustine).
Revert "Implement _ExtInt as an extended int type specifier."
Apr 17 2020, 10:48 AM
saugustine added a reverting change for rG61ba1481e200: Implement _ExtInt as an extended int type specifier.: rGa4b88c044980: Revert "Implement _ExtInt as an extended int type specifier.".
Apr 17 2020, 10:48 AM
saugustine added a comment to D73967: Implement _ExtInt as an extended int type specifier..

Reverted in a4b88c044980337bb14390be654fe76864aa60ec. Happy to approve an updated change.

Apr 17 2020, 10:47 AM · Restricted Project
saugustine added a comment to D73967: Implement _ExtInt as an extended int type specifier..

This change breaks the lldb build with

Apr 17 2020, 10:14 AM · Restricted Project

Apr 16 2020

saugustine added a comment to D77353: [WebAssembly] Add DW_OP_WASM_location_int.

Appreciate the quick fix. Thanks.

Apr 16 2020, 7:30 PM · debug-info, Restricted Project
saugustine added a comment to D77353: [WebAssembly] Add DW_OP_WASM_location_int.

The comparison on lines 81 and 82 of InputChunks.cpp is always true.

Apr 16 2020, 5:50 PM · debug-info, Restricted Project

Apr 15 2020

saugustine committed rGbf94c960071d: Write ignored output to stdout, so this test runs on read-only filesystems. (authored by saugustine).
Write ignored output to stdout, so this test runs on read-only filesystems.
Apr 15 2020, 10:57 AM
saugustine added a comment to D76848: [CodeGen][ARM] Error when writing to specific reserved registers in inline asm.

The test in this commit writes to a local file (which it ignores). This makes it so the test can't run on a read-only file system. I have committed bf94c960071d338b7157ac7dee8120df50d5600f to fix.

Apr 15 2020, 10:56 AM · Restricted Project

Apr 13 2020

saugustine committed rG215e6bfcfb5a: Use a temporary file, to allow running on read-only file systems (authored by saugustine).
Use a temporary file, to allow running on read-only file systems
Apr 13 2020, 2:09 PM
saugustine added a comment to D77580: [llvm-objdump] Fix incomplete relocation output for -D -r mode .

As written, this test writes to the current directory, which assumes that the current directory is writable, but that is not true in all builds. I have committed 215e6bfcfb5af7713ec348f679c7be4d2f32dc82 to fix.

Apr 13 2020, 2:08 PM · Restricted Project

Apr 7 2020

saugustine accepted D77679: [libunwind] Fix UB in EHHeaderParser::findFDE.

Nice catch. Probably should wait for an unwind owner to accept though.

Apr 7 2020, 2:42 PM · Unknown Object (Project), Restricted Project

Apr 2 2020

saugustine added a comment to D77099: Exit unwinding early when at the top of the stack and additional info won't be found..

Don't know why Phab thinks compnerd's accepting the revision March 1, at 9:54 left it in an unaccepted state. Happy to revert it if necessary, but I don't think so.

Apr 2 2020, 12:28 PM · Unknown Object (Project), Restricted Project
saugustine committed rG71fbd6e40632: Exit unwinding early when at the top of the stack and additional info won't be… (authored by saugustine).
Exit unwinding early when at the top of the stack and additional info won't be…
Apr 2 2020, 11:23 AM
saugustine closed D77099: Exit unwinding early when at the top of the stack and additional info won't be found..
Apr 2 2020, 11:23 AM · Unknown Object (Project), Restricted Project

Mar 31 2020

saugustine committed rG21d9d0855be1: New symbolizer option to print files relative to the compilation directory. (authored by saugustine).
New symbolizer option to print files relative to the compilation directory.
Mar 31 2020, 10:00 AM
saugustine closed D76733: New symbolizer option to print files relative to the compilation directory..
Mar 31 2020, 10:00 AM · Restricted Project

Mar 30 2020

saugustine added a comment to D77099: Exit unwinding early when at the top of the stack and additional info won't be found..

The harbormaster failures can't possibly be related. For example, the failure below has nothing whatever to do with this change.

Mar 30 2020, 4:57 PM · Unknown Object (Project), Restricted Project
saugustine updated the diff for D76733: New symbolizer option to print files relative to the compilation directory..

Thanks for the comments, I think this catches them all.

Mar 30 2020, 4:24 PM · Restricted Project
saugustine added a comment to D76733: New symbolizer option to print files relative to the compilation directory..

updated for all comments.

Mar 30 2020, 4:24 PM · Restricted Project
saugustine added reviewers for D77099: Exit unwinding early when at the top of the stack and additional info won't be found.: jgorbe, mstorsjo, compnerd, miyuki, mclow.lists.
Mar 30 2020, 3:51 PM · Unknown Object (Project), Restricted Project
saugustine created D77099: Exit unwinding early when at the top of the stack and additional info won't be found..
Mar 30 2020, 3:51 PM · Unknown Object (Project), Restricted Project

Mar 26 2020

saugustine updated the diff for D76733: New symbolizer option to print files relative to the compilation directory..

clang-format fix

Mar 26 2020, 3:47 PM · Restricted Project
saugustine updated the diff for D76733: New symbolizer option to print files relative to the compilation directory..

Add missing file.

Mar 26 2020, 2:42 PM · Restricted Project
saugustine updated the diff for D76733: New symbolizer option to print files relative to the compilation directory..

Add documentation and otherwise address upstream comments.

Mar 26 2020, 2:08 PM · Restricted Project
saugustine committed rG6c6fba8808e6: Correctly handle using foo = std::foo inside namespaces. (authored by saugustine).
Correctly handle using foo = std::foo inside namespaces.
Mar 26 2020, 11:26 AM
saugustine closed D76816: Correctly handle using foo = std::foo inside namespaces..
Mar 26 2020, 11:25 AM · Restricted Project

Mar 25 2020

saugustine created D76816: Correctly handle using foo = std::foo inside namespaces..
Mar 25 2020, 6:25 PM · Restricted Project
saugustine added a reviewer for D76816: Correctly handle using foo = std::foo inside namespaces.: EricWF.
Mar 25 2020, 6:25 PM · Restricted Project

Mar 24 2020

saugustine updated the diff for D76733: New symbolizer option to print files relative to the compilation directory..

Remove short option. It only existed to be parallel to "-s", but conflicts
with options in gnu addr2line.

Mar 24 2020, 5:30 PM · Restricted Project
saugustine added a comment to D76733: New symbolizer option to print files relative to the compilation directory..

The clang-format recommendations would change the new options to not match the format of old ones. Not sure the best thing to do there.

Mar 24 2020, 3:39 PM · Restricted Project
saugustine added a comment to D76733: New symbolizer option to print files relative to the compilation directory..

This change looks good to me. My mere concern is about the option naming. In GNU addr2line, -r has a different meaning, and llvm-symbolizer can be used as llvm-addr2line (it is a crunched executable). If it does not cause too much inconvenience, I will hope we can make llvm-symbolizer and llvm-addr2line have consistent option names.

Mar 24 2020, 3:39 PM · Restricted Project
saugustine created D76733: New symbolizer option to print files relative to the compilation directory..
Mar 24 2020, 3:06 PM · Restricted Project

Mar 20 2020

saugustine committed rG5de4ba1770f4: Cleanup the plumbing for DILineInfoSpecifier. [NFC - Try 2] (authored by saugustine).
Cleanup the plumbing for DILineInfoSpecifier. [NFC - Try 2]
Mar 20 2020, 10:51 AM

Mar 19 2020

saugustine committed rG6343526d6400: Revert "Cleanup the plumbing for DILineInfoSpecifier. [NFC]" (authored by saugustine).
Revert "Cleanup the plumbing for DILineInfoSpecifier. [NFC]"
Mar 19 2020, 5:35 PM
saugustine added a reverting change for rG98ff6eb679cd: Cleanup the plumbing for DILineInfoSpecifier. [NFC]: rG6343526d6400: Revert "Cleanup the plumbing for DILineInfoSpecifier. [NFC]".
Mar 19 2020, 5:35 PM
saugustine committed rG98ff6eb679cd: Cleanup the plumbing for DILineInfoSpecifier. [NFC] (authored by saugustine).
Cleanup the plumbing for DILineInfoSpecifier. [NFC]
Mar 19 2020, 5:02 PM
saugustine closed D76394: Cleanup the plumbing for DILineInfoSpecifier. [NFC].
Mar 19 2020, 5:02 PM · Restricted Project

Mar 18 2020

saugustine created D76394: Cleanup the plumbing for DILineInfoSpecifier. [NFC].
Mar 18 2020, 4:19 PM · Restricted Project

Mar 12 2020

saugustine committed rG3758b858ef3b: Only run frameheader_cache_test.pass.cpp on x86_64. (authored by saugustine).
Only run frameheader_cache_test.pass.cpp on x86_64.
Mar 12 2020, 6:21 PM
saugustine committed rG3eef58991b26: Add a catch-all else case so any unanticipated configs pass this test. (authored by saugustine).
Add a catch-all else case so any unanticipated configs pass this test.
Mar 12 2020, 11:59 AM
saugustine committed rGc53c2058ffb8: Cache uwnind frame headers as they are found. (authored by saugustine).
Cache uwnind frame headers as they are found.
Mar 12 2020, 11:24 AM
saugustine closed D75954: Cache uwnind frame headers as they are found..
Mar 12 2020, 11:24 AM · Restricted Project, Restricted Project
saugustine updated the diff for D75954: Cache uwnind frame headers as they are found..

fix comment

Mar 12 2020, 9:13 AM · Restricted Project, Restricted Project

Mar 11 2020

saugustine updated the diff for D75954: Cache uwnind frame headers as they are found..

Correct the "do not run" cases to match the logic in AddressSpace.hpp.

Mar 11 2020, 3:32 PM · Restricted Project, Restricted Project
saugustine updated the diff for D75954: Cache uwnind frame headers as they are found..

Update test to only run when relevant.

Mar 11 2020, 3:22 PM · Restricted Project, Restricted Project
saugustine updated the diff for D75954: Cache uwnind frame headers as they are found..

Add tests, and minor formatting changes.

Mar 11 2020, 3:20 PM · Restricted Project, Restricted Project
saugustine added a comment to D75954: Cache uwnind frame headers as they are found..

I think it looks good now. My only issue is that it seems to rely on a couple of glibc-specific features: glibc modifying the fields adds and subs in dl_phdr_info when loading/unloading libraries (which this patch uses to know when to invalidate the cache), and dl_iterate_phdr holding a lock (which the patch relies on to avoid races while accessing the cache). What other libc implementations do we support? Do they share these behaviors we rely on here?

Mar 11 2020, 11:52 AM · Restricted Project, Restricted Project
saugustine updated the diff for D75954: Cache uwnind frame headers as they are found..

Add missing comment.

Mar 11 2020, 10:45 AM · Restricted Project, Restricted Project
saugustine committed rG8ffdabdb61e1: Lazily save initialState of registers during unwind. (authored by saugustine).
Lazily save initialState of registers during unwind.
Mar 11 2020, 10:14 AM
saugustine closed D75692: Lazily save initialState of registers during unwind..
Mar 11 2020, 10:14 AM · Restricted Project
saugustine added inline comments to D75954: Cache uwnind frame headers as they are found..
Mar 11 2020, 10:09 AM · Restricted Project, Restricted Project
saugustine updated the diff for D75954: Cache uwnind frame headers as they are found..

Update for upstream comments.

Mar 11 2020, 10:09 AM · Restricted Project, Restricted Project

Mar 10 2020

saugustine added reviewers for D75954: Cache uwnind frame headers as they are found.: miyuki, mstorsjo, compnerd, jgorbe.
Mar 10 2020, 2:13 PM · Restricted Project, Restricted Project
saugustine added a comment to D75954: Cache uwnind frame headers as they are found..

I have some tests nearing completion, but want to get started with the feedback now.

Mar 10 2020, 2:13 PM · Restricted Project, Restricted Project
saugustine accepted D75890: [libunwind] Remove __FILE__ and __LINE__ from error reporting.

This looks fine to me.

Mar 10 2020, 1:06 PM · Restricted Project, Restricted Project
saugustine created D75954: Cache uwnind frame headers as they are found..
Mar 10 2020, 12:33 PM · Restricted Project, Restricted Project
saugustine committed rGd67cf7a0a970: Rework findUnwindSectionsByPhdr to be more optimal. (authored by saugustine).
Rework findUnwindSectionsByPhdr to be more optimal.
Mar 10 2020, 10:54 AM
saugustine closed D75781: Rework findUnwindSectionsByPhdr to be more optimal..
Mar 10 2020, 10:53 AM · Restricted Project

Mar 9 2020

saugustine updated the diff for D75781: Rework findUnwindSectionsByPhdr to be more optimal..

Update for upstream comments

Mar 9 2020, 5:50 PM · Restricted Project
saugustine added inline comments to D75781: Rework findUnwindSectionsByPhdr to be more optimal..
Mar 9 2020, 5:49 PM · Restricted Project
saugustine added inline comments to D75781: Rework findUnwindSectionsByPhdr to be more optimal..
Mar 9 2020, 4:45 PM · Restricted Project
saugustine updated the diff for D75692: Lazily save initialState of registers during unwind..

Switch boolean to a more clear enum to make call site clearer.

Mar 9 2020, 4:45 PM · Restricted Project
saugustine added a reviewer for D75692: Lazily save initialState of registers during unwind.: jgorbe.
Mar 9 2020, 2:35 PM · Restricted Project
saugustine updated the summary of D75781: Rework findUnwindSectionsByPhdr to be more optimal..
Mar 9 2020, 12:25 PM · Restricted Project
saugustine updated the diff for D75692: Lazily save initialState of registers during unwind..

Move initialization to in class.

Mar 9 2020, 11:21 AM · Restricted Project