Page MenuHomePhabricator

ikudrin (Igor Kudrin)
User

Projects

User does not belong to any projects.

User Details

User Since
Sep 6 2015, 10:51 PM (293 w, 1 d)

Recent Activity

Fri, Apr 16

ikudrin added inline comments to D75485: Support DW_FORM_strx* in llvm-dwp..
Fri, Apr 16, 5:13 AM · Restricted Project

Mar 10 2021

ikudrin accepted D98383: [Debug-Info] Add names for the debug line prologue..

LGTM, thanks!

Mar 10 2021, 9:24 PM · Restricted Project

Mar 7 2021

ikudrin accepted D98111: [CMake][compiler-rt] Use copying instead of symlinking for LSE builtins on non-Unix-likes.

The fix resolves the issue on my environment. Thank you for the quick response!

Mar 7 2021, 8:37 PM · Restricted Project

Mar 5 2021

ikudrin added a comment to D93278: Re-apply "[CMake][compiler-rt][AArch64] Avoid preprocessing LSE builtins separately".

This change causes build failures in my Windows environment.

Mar 5 2021, 5:41 AM · Restricted Project

Feb 22 2021

ikudrin added inline comments to D96409: [debug-info] refactor emitDwarfUnitLength.
Feb 22 2021, 5:07 AM · Restricted Project, debug-info
ikudrin accepted D95998: [Debug-Info] [NFC] use emitDwarfUnitLength to handle debug line section unit length field.

Looks good, thanks!

Feb 22 2021, 2:39 AM · debug-info, Restricted Project
ikudrin accepted D96409: [debug-info] refactor emitDwarfUnitLength.

LGTM, thanks!

Feb 22 2021, 2:20 AM · Restricted Project, debug-info

Feb 19 2021

ikudrin added a comment to D96409: [debug-info] refactor emitDwarfUnitLength.

More in the sense that it limits the flexibility of calling code without any real profit. For example, the caller now cannot create the label in advance.

Any particular benefit to being able to do that, though? I think this is similar to rolling "Lo" into the function - the caller now can't reuse that label, but it doesn't seem especially important/beneficial to be able to reuse it, compared to simplifying the API.

Why should a service class like MCStreamer define the architecture of its clients? It is just not its area of responsibility.

Feb 19 2021, 7:35 PM · Restricted Project, debug-info
ikudrin added a comment to D96865: [Driver] Honor "-gdwarf-N" at any position for assembler sources.

Do you have any suggestions on why we would start seeing above failures after you commit? Was passing before.

Feb 19 2021, 6:20 PM · debug-info, Restricted Project, Restricted Project
ikudrin added a comment to D96409: [debug-info] refactor emitDwarfUnitLength.

At some point it'd probably be good to include a second (probably as the first parameter) to this function to use as a prefix for the label names to make them more informative/easier to read/modify/etc.

Maybe it'll make sense to do that in this patch. There will be fewer changes in the tests.

Feb 19 2021, 2:35 AM · Restricted Project, debug-info
ikudrin added a comment to D96409: [debug-info] refactor emitDwarfUnitLength.

(Moving the thread, as it's more suitable here)

What if the function created and returned Hi rather than taking it as an argument?

Hmm, I can't say I'm a vivid fan of this change. All methods of MCStremer instruct it to do something, i.e. it roles as just a performer. With this change, it now requests a caller to do some action. It looks like inverting customer vs. performer roles for this particular case.

In the sense that it now requests the user emit the end label at some point?

More in the sense that it limits the flexibility of calling code without any real profit. For example, the caller now cannot create the label in advance. For instance, Dwarf5AccelTableWriter::ContributionEnd was initialized when the object of that class was created. Simple and solid. Now the initialization is postponed until a method of another class, Dwarf5AccelTableWriter::Header is called. The tracking is a bit more challenging now.

Feb 19 2021, 2:25 AM · Restricted Project, debug-info

Feb 17 2021

ikudrin committed rGa0c9ec1f5e70: [Driver] Honor "-gdwarf-N" at any position for assembler sources (authored by ikudrin).
[Driver] Honor "-gdwarf-N" at any position for assembler sources
Feb 17 2021, 7:37 PM
ikudrin closed D96865: [Driver] Honor "-gdwarf-N" at any position for assembler sources.
Feb 17 2021, 7:37 PM · debug-info, Restricted Project, Restricted Project
ikudrin added a comment to D96865: [Driver] Honor "-gdwarf-N" at any position for assembler sources.

Thanks!

Feb 17 2021, 4:28 PM · debug-info, Restricted Project, Restricted Project
ikudrin requested review of D96865: [Driver] Honor "-gdwarf-N" at any position for assembler sources.
Feb 17 2021, 6:53 AM · debug-info, Restricted Project, Restricted Project
ikudrin added a comment to D95998: [Debug-Info] [NFC] use emitDwarfUnitLength to handle debug line section unit length field.

What if the function created and returned Hi rather than taking it as an argument?

Feb 17 2021, 4:20 AM · debug-info, Restricted Project
ikudrin added a comment to D96710: [DebugInfo] Improve tests for AsmPrinter::emitDwarfUnitLength().

@ikudrin Hi, we made another change for emitDwarfUnitLength in D96409, I think this patch needs to be further improved for Hi parameter. Let me know if you need me to update this patch. Thanks.

Feb 17 2021, 3:29 AM · debug-info, Restricted Project
ikudrin committed rG72eee60b2451: [Driver] Support -gdwarf64 for assembly files (authored by ikudrin).
[Driver] Support -gdwarf64 for assembly files
Feb 17 2021, 2:06 AM
ikudrin committed rGaa842896299b: [DebugInfo] Keep the DWARF64 flag in the module metadata (authored by ikudrin).
[DebugInfo] Keep the DWARF64 flag in the module metadata
Feb 17 2021, 2:06 AM
ikudrin closed D96783: [Driver] Support -gdwarf64 for assembly files.
Feb 17 2021, 2:06 AM · debug-info, Restricted Project, Restricted Project
ikudrin closed D96597: [DebugInfo] Keep the DWARF64 flag in the module metadata.
Feb 17 2021, 2:06 AM · debug-info, Restricted Project, Restricted Project

Feb 16 2021

ikudrin added inline comments to D95998: [Debug-Info] [NFC] use emitDwarfUnitLength to handle debug line section unit length field.
Feb 16 2021, 10:59 PM · debug-info, Restricted Project
ikudrin added a comment to D96710: [DebugInfo] Improve tests for AsmPrinter::emitDwarfUnitLength().

Sure, sounds OK. Though a lot of this is probably OK/already tested by assembly level tests, I'd expect.

Feb 16 2021, 10:52 PM · debug-info, Restricted Project
ikudrin added inline comments to D96783: [Driver] Support -gdwarf64 for assembly files.
Feb 16 2021, 10:38 PM · debug-info, Restricted Project, Restricted Project
ikudrin added a comment to D96144: [ELF] Add --dwarf32-before-dwarf64 to sort DWARF32 input sections before DWARF64.

Ping. What should be done to proceed with this?

Feb 16 2021, 7:46 AM · debug-info, Restricted Project, lld
ikudrin added inline comments to D96409: [debug-info] refactor emitDwarfUnitLength.
Feb 16 2021, 7:44 AM · Restricted Project, debug-info
ikudrin requested review of D96783: [Driver] Support -gdwarf64 for assembly files.
Feb 16 2021, 6:40 AM · debug-info, Restricted Project, Restricted Project

Feb 15 2021

ikudrin added a comment to D96409: [debug-info] refactor emitDwarfUnitLength.

@shchenz, please take a look at D96710. When it is accepted, you can integrate it here or commit it separately within the whole set.

Feb 15 2021, 7:12 AM · Restricted Project, debug-info
ikudrin requested review of D96710: [DebugInfo] Improve tests for AsmPrinter::emitDwarfUnitLength().
Feb 15 2021, 7:06 AM · debug-info, Restricted Project
ikudrin added a comment to D95998: [Debug-Info] [NFC] use emitDwarfUnitLength to handle debug line section unit length field.

I guess we can simplify that even further by emitting the temporary symbol right after the "length of the prologue" field. In that case, PreHeaderLengthBytes will not be needed at all.

Feb 15 2021, 6:43 AM · debug-info, Restricted Project
ikudrin added a comment to D96597: [DebugInfo] Keep the DWARF64 flag in the module metadata.

Thank you, @dblaikie!

Feb 15 2021, 6:20 AM · debug-info, Restricted Project, Restricted Project
ikudrin added inline comments to D96409: [debug-info] refactor emitDwarfUnitLength.
Feb 15 2021, 1:51 AM · Restricted Project, debug-info

Feb 12 2021

ikudrin requested review of D96597: [DebugInfo] Keep the DWARF64 flag in the module metadata.
Feb 12 2021, 5:50 AM · debug-info, Restricted Project, Restricted Project

Feb 10 2021

ikudrin added a comment to D94668: [debug-info] [NFC] add is-a(isa<>) support for MCStreamer.

Do you have any idea about how to do this? @ikudrin My requirement here is: I need to add the endSymbol for asm streamer at the end of .text section to indicate the end of the text section.
Object streamer can easily do this by calling MCSymbol *endSection(Section) in emitDwarfLineTable, but this does not work for asm streamer. When we are generating assembly output for .debug_line section, we can not go back to .text section and emit an end symbol there. I think this is not supported.

Feb 10 2021, 2:22 AM · Restricted Project
ikudrin added a comment to D95998: [Debug-Info] [NFC] use emitDwarfUnitLength to handle debug line section unit length field.

The idea was to remove the Lo argument from MCStreamer::emitDwarfUnitLength(Hi, Lo, Comment) so that it becomes just MCStreamer::emitDwarfUnitLength(Hi, Comment). A temporary symbol can be created and emitted inside the method. And that should be a separate NFC patch.

We need to differentiate the temp symbol names for different debug sections like debug info, debug pubnames, debug line to make a more readable assembly output. That's the first reason I put the temp symbol created outside.

Feb 10 2021, 1:39 AM · debug-info, Restricted Project
ikudrin added a comment to D95998: [Debug-Info] [NFC] use emitDwarfUnitLength to handle debug line section unit length field.

The idea was to remove the Lo argument from MCStreamer::emitDwarfUnitLength(Hi, Lo, Comment) so that it becomes just MCStreamer::emitDwarfUnitLength(Hi, Comment). A temporary symbol can be created and emitted inside the method. And that should be a separate NFC patch.

Feb 10 2021, 12:45 AM · debug-info, Restricted Project
ikudrin added a comment to D94668: [debug-info] [NFC] add is-a(isa<>) support for MCStreamer.

Gentle ping...

We still need this patch for D95518 for lib/Target/PowerPC/PPCAsmPrinter.cpp file. We need to emit the end symbol for asmstreamer to tell where is the end of the text section.

Feb 10 2021, 12:22 AM · Restricted Project

Feb 9 2021

ikudrin added a comment to D95998: [Debug-Info] [NFC] use emitDwarfUnitLength to handle debug line section unit length field.

There is no need to create a new MCStreamer::emitDwarfLineUnitLength() method and make all the dependencies public. It is better to just use the already existed MCStreamer::emitDwarfUnitLength(). All you need to do is to create a temporary symbol and emit it right after calling emitDwarfUnitLength().

Feb 9 2021, 5:00 AM · debug-info, Restricted Project
ikudrin added a comment to D96144: [ELF] Add --dwarf32-before-dwarf64 to sort DWARF32 input sections before DWARF64.

I've re-read a fair amount of the previous llvm-dev thread.
One goal of doing a patch like this was to collect some performance data; do we have that? In particular, performance data comparing link times with the option default-on versus default-off, so we can determine whether the difference is small enough that we should just do this processing unconditionally.

Feb 9 2021, 3:17 AM · debug-info, Restricted Project, lld

Feb 8 2021

ikudrin added a comment to D96144: [ELF] Add --dwarf32-before-dwarf64 to sort DWARF32 input sections before DWARF64.

There were so many ideas in the discussion that it looked like they not going to converge. The idea of waiting for v6 does not resolve the issue for existing standards, just postpones the possible solution for years with a yet unknown result. Adding new section names, flags, types, etc., might be promising at the first glance but requires updating lots of tools, including tools in different toolchains, which complicates achieving the result even further. That is why I feel it is necessary to illustrate my proposal with a complete patch, which is aimed to show that a simple, efficient, and standard-compliant solution is possible.

Feb 8 2021, 5:53 AM · debug-info, Restricted Project, lld

Feb 5 2021

ikudrin requested review of D96144: [ELF] Add --dwarf32-before-dwarf64 to sort DWARF32 input sections before DWARF64.
Feb 5 2021, 7:31 AM · debug-info, Restricted Project, lld

Feb 4 2021

ikudrin accepted D95932: [Debug-Info] [NFC] move emitDwarfUnitLength to MCStreamer class.

LGTM but please postpone committing it until the whole set of patches is completed so we can have a full image of the changes.

Feb 4 2021, 2:53 AM · debug-info, Restricted Project

Feb 3 2021

ikudrin added inline comments to D95932: [Debug-Info] [NFC] move emitDwarfUnitLength to MCStreamer class.
Feb 3 2021, 11:33 PM · debug-info, Restricted Project
ikudrin added inline comments to D95932: [Debug-Info] [NFC] move emitDwarfUnitLength to MCStreamer class.
Feb 3 2021, 10:31 PM · debug-info, Restricted Project

Feb 2 2021

ikudrin added a comment to D95200: [test] Add basic _Unwind_ForcedUnwind + exception tests.

Seems OK. Something similar is in exception_object_alignment.pass.cpp.

Feb 2 2021, 10:49 PM · Restricted Project
ikudrin added a comment to D95200: [test] Add basic _Unwind_ForcedUnwind + exception tests.

I am not sure libunwind's ARMEH implementation has working _Unwind_ForcedUnwind so looks like the best is to unsupport arm. How to do that?

Feb 2 2021, 10:36 PM · Restricted Project
ikudrin added a comment to D94670: [DebugInfo][NFC] add a new DIE type to represent label + offset.

It looks like my idea was not well-thought.

Feb 2 2021, 2:04 AM · Restricted Project

Feb 1 2021

ikudrin added a comment to D94670: [DebugInfo][NFC] add a new DIE type to represent label + offset.

Is it possible to use a .set pseudo-op to define a symbol that can be referenced from other sections?

See https://reviews.llvm.org/D94670#2501150. It seems to work. Do you have a suggestion on how to make use of it from the MC layer here?

Feb 1 2021, 2:36 AM · Restricted Project

Jan 29 2021

ikudrin added inline comments to D95518: [Debug-Info][XCOFF] support dwarf for XCOFF for assembly output.
Jan 29 2021, 8:49 PM · debug-info, Restricted Project
ikudrin added a comment to D94670: [DebugInfo][NFC] add a new DIE type to represent label + offset.

The comment was originally sent to D94668, but it looks like it suits here better.

Jan 29 2021, 8:30 PM · Restricted Project
ikudrin added a comment to D94668: [debug-info] [NFC] add is-a(isa<>) support for MCStreamer.
Jan 29 2021, 8:00 AM · Restricted Project

Jan 15 2021

ikudrin added a comment to D94668: [debug-info] [NFC] add is-a(isa<>) support for MCStreamer.

Is it possible to encapsulate the target-specific logic into some derived classes from MCStreamer so that MCStreamer itself and DWARF emitter classes work through a common interface and have no target-specific adjustments?

Jan 15 2021, 5:18 AM · Restricted Project
ikudrin added a comment to D94670: [DebugInfo][NFC] add a new DIE type to represent label + offset.

I suppose that you describe how .dwsect pseudo-op works. That is quite interesting, why they designed the feature to work that way. Is it recommended to reference debug sections through the label minus the length field size (4 or 12) or they provide some means to simplify the calculation? How an assembler output of their own compiler looks like?

Jan 15 2021, 5:18 AM · Restricted Project
ikudrin committed rG78036360573c: [libcxx testing] Fix UB in tests for std::lock_guard (authored by ikudrin).
[libcxx testing] Fix UB in tests for std::lock_guard
Jan 15 2021, 1:12 AM
ikudrin closed D94656: [libcxx testing] Fix UB in tests for std::lock_guard.
Jan 15 2021, 1:12 AM · Restricted Project

Jan 14 2021

ikudrin updated the diff for D94656: [libcxx testing] Fix UB in tests for std::lock_guard.

Trying to force CI to run.

Jan 14 2021, 6:47 PM · Restricted Project
ikudrin added a project to D94656: [libcxx testing] Fix UB in tests for std::lock_guard: Restricted Project.
Jan 14 2021, 5:23 PM · Restricted Project
ikudrin updated the diff for D94656: [libcxx testing] Fix UB in tests for std::lock_guard.
  • Updated as suggested. Thanks, @ldionne!
Jan 14 2021, 8:04 AM · Restricted Project
ikudrin added inline comments to D94670: [DebugInfo][NFC] add a new DIE type to represent label + offset.
Jan 14 2021, 7:31 AM · Restricted Project
ikudrin added a comment to D94656: [libcxx testing] Fix UB in tests for std::lock_guard.

The current code violates the C++ standard, which says (in different sections, depending on the version):

Jan 14 2021, 4:31 AM · Restricted Project
ikudrin requested review of D94656: [libcxx testing] Fix UB in tests for std::lock_guard.
Jan 14 2021, 12:16 AM · Restricted Project

Dec 20 2020

ikudrin added inline comments to D90507: [Driver] Add DWARF64 flag: -gdwarf64.
Dec 20 2020, 9:05 PM · Restricted Project

Nov 18 2020

ikudrin added inline comments to D90507: [Driver] Add DWARF64 flag: -gdwarf64.
Nov 18 2020, 9:46 PM · Restricted Project

Nov 16 2020

ikudrin added a comment to D90507: [Driver] Add DWARF64 flag: -gdwarf64.

It looks like lld/test/COFF/lto-new-pass-manager.ll.obj was added to the patch by accident and should be removed.

Nov 16 2020, 8:19 PM · Restricted Project

Nov 13 2020

ikudrin added a comment to D91404: RFC: [ELF] Add --dwarf32-before-dwarf64 to place DWARF32 input sections before DWARF64.

Well, this is a bit different from my original idea but is an overall good heuristic for many of the debug sections. It works for .debug_info, which is one of the biggest sections; it does not work for .debug_line, though, which is not that big as .debug_info, but potentially might become problematic in the (distant) future; it also does not work for .debug_abbrev, .debug_addr, .debug_ranges, and some others, which are usually not that big. However, the patch should be extended to support .debug_str, which can be even larger than .debug_info.

Nov 13 2020, 5:33 AM · Restricted Project

Nov 6 2020

ikudrin added a comment to D87011: [DebugInfo] Add the -dwarf64 switch to llc and other internal tools (4/19)..

At least in LLD, it's not quite as simple as being added after the user's code: if a library appears on the link line it will be included in the output order as soon as it is determined it is needed. Thus if you have have three modules 1.o, 2.o, and 3.o, with 3.o in an archive 3.a and 1.o requiring 3.o, you end up with an output order of 1.o 3.o 2.o if the input order was 1.o 3.a 2.o or 3.a 1.o 2.o or an output order of 1.o 2.o 3.o if the input order was 1.o 2.o 3.a. In fact, with use of the --undefined linker switch, you can even force 3.o to appear first.

I accept using --undefined or rearranging the command-line order is less than ideal, but I'm really not convinced LLD should have any place in parsing the DWARF to determine output order. Furthermore, it's not even a reliable solution - if the objects built with DWARF32 (potentially all of which might have come from libraries) are large enough, no amount of reordering will fix the behaviour. I think users who need DWARF64 in their libraries are just going to have to request DWARF64 versions of the libraries, if the --undefined and reordering command line options are insufficient.

Nov 6 2020, 3:09 AM · debug-info, Restricted Project
ikudrin added a comment to D87011: [DebugInfo] Add the -dwarf64 switch to llc and other internal tools (4/19)..

I wouldn't expect LLD to need to do anything specific for DWARF64 support. It should generally speaking be treating the sections as opaque, in my opinion, and treat them no differently to other sections. If a user is mixing DWARF32 and DWARF64, then I'd say it's on their heads if relocations can't reach (just the same as it is if they're using DWARF32 but really need DWARF64). Any interactions LLD does have with the contents of the DWARF sections should be controlled via the DebugInfo library, and therefore if that library works for DWARF64, LLD doesn't need any special handling.

Nov 6 2020, 1:32 AM · debug-info, Restricted Project

Nov 5 2020

ikudrin added a comment to D90507: [Driver] Add DWARF64 flag: -gdwarf64.
  1. The patch needs tests to check the added functionality.
  2. DWARF64 can be generated only for a limited number of targets. There should be diagnostics for invalid switch combinations to prevent misuse. @MaskRay mentioned that in the patch for llc, D87011#2254749, but that makes a lot more sense for user-level tools.
Nov 5 2020, 9:40 PM · Restricted Project
ikudrin added a comment to D87011: [DebugInfo] Add the -dwarf64 switch to llc and other internal tools (4/19)..

I suppose that it would be helpful to arrange debugging information sections so that DWARF64 comes after DWARF32, otherwise, some 32-bit relocations in the 32-bit info could not be resolved. But that idea might be a bit controversial because usually debugging information is expected to have the same order as the sections it refers to.

Nov 5 2020, 7:21 PM · debug-info, Restricted Project

Nov 4 2020

ikudrin added a comment to D87011: [DebugInfo] Add the -dwarf64 switch to llc and other internal tools (4/19)..

To generate 64-bit debugging info, there should be enough to pass the switch through CLANG, right. Apart from that, we will probably need some compatibility checks so that using the switch in unsupported cases prints out diagnostics. There are also some improvements on the LLD side which are better to be done to support extremely large debugging information. Not all our tools fully support DWARF64 yet, etc.

Nov 4 2020, 9:07 PM · debug-info, Restricted Project

Oct 28 2020

ikudrin added a comment to D87011: [DebugInfo] Add the -dwarf64 switch to llc and other internal tools (4/19)..

The switch is implemented only internally in LLVM. There is still some work to be done to enable producing 64-bit debugging info in clang, but I strayed a bit for another task. Hope to come back later this year.

Oct 28 2020, 7:48 PM · debug-info, Restricted Project

Oct 27 2020

ikudrin added a comment to D87011: [DebugInfo] Add the -dwarf64 switch to llc and other internal tools (4/19)..

@ikudrin To clarify this will emit R_X86_64_64 bit relocations for .debug_info on 64 bit platform, correct?

Oct 27 2020, 10:16 PM · debug-info, Restricted Project

Oct 23 2020

ikudrin added a comment to D88877: [ELF] Drop --whole-archive before processing dependent libraries..

I am still concerned that we go down this route (workaround in the linker rather than teach the user to add --no-whole-archive just because of convenience adding logic in the linker). Among --push-state pushed states, --as-needed and -Bstatic may be states users want to keep for the dependent libraries.

Oct 23 2020, 8:53 PM · Restricted Project, lld

Oct 22 2020

ikudrin updated the diff for D88877: [ELF] Drop --whole-archive before processing dependent libraries..
Oct 22 2020, 11:46 PM · Restricted Project, lld

Oct 21 2020

ikudrin added inline comments to D88877: [ELF] Drop --whole-archive before processing dependent libraries..
Oct 21 2020, 8:03 AM · Restricted Project, lld
ikudrin updated the diff for D88877: [ELF] Drop --whole-archive before processing dependent libraries..

I think I agree that build systems might not easily be able to arrange for appending options to the end of the linker command line. Therefore, for the "dependent libraries" feature (I changed the name in https://reviews.llvm.org/D60274 because autolinking was too general) it would be reasonable to reset any temporal options that the build system would otherwise have to handle by appending an argument to the very end of the command line (e.g. --whole-archives). Where appropriate we can create new order-independent options to apply the options that are reset to the dependent libraries. This seems like an improvement over what we have now.

Oct 21 2020, 8:03 AM · Restricted Project, lld

Oct 19 2020

ikudrin committed rG1329944c221c: [LTO][Legacy] Eliminate an unused member of llvm::LTOModule. NFC. (authored by ikudrin).
[LTO][Legacy] Eliminate an unused member of llvm::LTOModule. NFC.
Oct 19 2020, 11:35 PM
ikudrin closed D89472: [LTO][Legacy] Eliminate an unused member of llvm::LTOModule. NFC..
Oct 19 2020, 11:35 PM · Restricted Project

Oct 16 2020

ikudrin added a comment to D88877: [ELF] Drop --whole-archive before processing dependent libraries..

Hmm, that starts looking like a discussion for the mailing list, but I guess that all interested people are already here and the conversation will be better preserved for further references if being linked to the review.

Oct 16 2020, 3:03 AM · Restricted Project, lld

Oct 15 2020

ikudrin added reviewers for D89472: [LTO][Legacy] Eliminate an unused member of llvm::LTOModule. NFC.: pcc, steven_wu.
Oct 15 2020, 8:54 PM · Restricted Project
ikudrin requested review of D89472: [LTO][Legacy] Eliminate an unused member of llvm::LTOModule. NFC..
Oct 15 2020, 7:58 AM · Restricted Project
ikudrin added inline comments to D88877: [ELF] Drop --whole-archive before processing dependent libraries..
Oct 15 2020, 12:50 AM · Restricted Project, lld
ikudrin updated the diff for D88877: [ELF] Drop --whole-archive before processing dependent libraries..
Oct 15 2020, 12:49 AM · Restricted Project, lld

Oct 14 2020

ikudrin added a comment to D88877: [ELF] Drop --whole-archive before processing dependent libraries..

Firstly, is this a real problem?

Oct 14 2020, 11:14 PM · Restricted Project, lld
ikudrin added a comment to D88877: [ELF] Drop --whole-archive before processing dependent libraries..

Gently ping. Is there anything I can improve in the fix so that it is accepted?

Oct 14 2020, 8:42 AM · Restricted Project, lld

Oct 8 2020

ikudrin added a comment to D88944: [ELF] Skip repeated libraries..

I've made some tests and speculatively estimate the saving up to several seconds for a relatively large library (about 100000 symbols) which is referenced several hundred times. For me, that is noticeable.

Oct 8 2020, 6:59 AM · lld, Restricted Project

Oct 6 2020

ikudrin added a comment to D88944: [ELF] Skip repeated libraries..

I am skeptical about the change.

  • The duplicated libraries are usually indicator of user errors or build system brittleness.
Oct 6 2020, 11:59 PM · lld, Restricted Project
ikudrin updated the diff for D88877: [ELF] Drop --whole-archive before processing dependent libraries..
  • Simplified the comment.
Oct 6 2020, 11:26 PM · Restricted Project, lld
ikudrin requested review of D88944: [ELF] Skip repeated libraries..
Oct 6 2020, 10:04 PM · lld, Restricted Project
ikudrin updated the diff for D88877: [ELF] Drop --whole-archive before processing dependent libraries..
  • Fixed the comment.
Oct 6 2020, 9:45 PM · Restricted Project, lld
ikudrin added inline comments to D88877: [ELF] Drop --whole-archive before processing dependent libraries..
Oct 6 2020, 9:39 PM · Restricted Project, lld
ikudrin requested review of D88877: [ELF] Drop --whole-archive before processing dependent libraries..
Oct 6 2020, 2:12 AM · Restricted Project, lld

Sep 21 2020

ikudrin committed rG474d527c28f4: [clang] Fix a misleading variable name. NFC. (authored by ikudrin).
[clang] Fix a misleading variable name. NFC.
Sep 21 2020, 9:00 AM
ikudrin closed D88021: [clang] Fix a misleading variable name. NFC..
Sep 21 2020, 9:00 AM · Restricted Project, Restricted Project
ikudrin requested review of D88021: [clang] Fix a misleading variable name. NFC..
Sep 21 2020, 5:53 AM · Restricted Project, Restricted Project

Sep 16 2020

ikudrin committed rG027d47d1c7ce: [DebugInfo] Simplify DIEInteger::SizeOf(). (authored by ikudrin).
[DebugInfo] Simplify DIEInteger::SizeOf().
Sep 16 2020, 10:48 PM
ikudrin closed D85293: [DebugInfo] Simplify DIEInteger::SizeOf()..
Sep 16 2020, 10:48 PM · debug-info, Restricted Project
ikudrin added inline comments to D85293: [DebugInfo] Simplify DIEInteger::SizeOf()..
Sep 16 2020, 6:58 PM · debug-info, Restricted Project
ikudrin updated the diff for D85293: [DebugInfo] Simplify DIEInteger::SizeOf()..

D87008 added creating a test AsmPrinter to the CodeGen unit tests, so it is now possible to fix the DIEHashTest.MemberBlock so that it provides an instance to the tested class.

Sep 16 2020, 6:46 AM · debug-info, Restricted Project

Sep 14 2020

ikudrin committed rGa845ebd6333d: [DebugInfo] Make offsets of dwarf units 64-bit (19/19). (authored by ikudrin).
[DebugInfo] Make offsets of dwarf units 64-bit (19/19).
Sep 14 2020, 10:26 PM