Page MenuHomePhabricator

majnemer (David Majnemer)
User

Projects

User does not belong to any projects.

User Details

User Since
Apr 15 2013, 2:02 AM (450 w, 2 d)

Recent Activity

Sep 21 2021

majnemer added inline comments to D102817: [Windows SEH]: HARDWARE EXCEPTION HANDLING (MSVC -EHa) - Part 2.
Sep 21 2021, 10:18 AM · Restricted Project
majnemer accepted D109988: Teach InstCombine to eliminate malloc-realloc-free triplets..

LGTM

Sep 21 2021, 9:57 AM · Restricted Project

Jul 8 2021

majnemer added inline comments to D105672: [SimpleLoopUnswitch] Don't non-trivially unswitch loops with catchswitch exits.
Jul 8 2021, 9:33 PM · Restricted Project
majnemer requested changes to D105672: [SimpleLoopUnswitch] Don't non-trivially unswitch loops with catchswitch exits.
Jul 8 2021, 7:21 PM · Restricted Project

Jun 9 2021

majnemer added inline comments to D93470: [VP] Binary floating-point intrinsics..
Jun 9 2021, 12:02 AM · Restricted Project, Restricted Project

Jun 7 2021

majnemer added inline comments to D93470: [VP] Binary floating-point intrinsics..
Jun 7 2021, 12:19 AM · Restricted Project, Restricted Project

May 13 2021

majnemer accepted D102413: [MC] Add the ability to pass MCRegisterInfo to dump_pretty.

LGTM

May 13 2021, 1:30 PM · Restricted Project

Dec 7 2020

majnemer added inline comments to D92766: [XCOFF][AIX] Emit EH information in traceback table.
Dec 7 2020, 10:56 AM · Restricted Project

Dec 4 2020

majnemer added a comment to D92695: [X86ISelLowering] don't emit frame pointers for eflags intrinsics..

It is difficult remembering exactly what my thought process was way back when but the redzone calculation seems very likely...

Dec 4 2020, 5:21 PM · Restricted Project

Nov 19 2020

majnemer accepted D91313: Add MachineModuleInfo constructor with external MCContext.

LGTM

Nov 19 2020, 5:41 PM · Restricted Project

Oct 31 2020

majnemer added inline comments to D89628: Fix constant-folding of overflowing arithmetic ops on vectors.
Oct 31 2020, 9:27 AM · Restricted Project

Oct 30 2020

majnemer added inline comments to D88905: [Clang] Allow "ext_vector_type" applied to Booleans.
Oct 30 2020, 3:13 PM · Restricted Project

Aug 12 2020

majnemer accepted D85839: [InstCombine] Fix incorrect Modified status.

LGTM

Aug 12 2020, 1:16 PM · Restricted Project

Jul 30 2020

majnemer accepted D84896: [MC] Fix memory leak when allocating MCInst with bump allocator.

LGTM

Jul 30 2020, 11:02 AM · Restricted Project

Jul 8 2020

majnemer added inline comments to D83360: [InstSimplify] Remove select ?, undef, X -> X and select ?, X, undef -> X.
Jul 8 2020, 2:31 PM · Restricted Project, Restricted Project

Jun 3 2020

majnemer added inline comments to D80344: [Windows SEH]: HARDWARE EXCEPTION HANDLING (MSVC -EHa) - Part 1.
Jun 3 2020, 6:45 PM · Restricted Project, Restricted Project

May 14 2020

majnemer added inline comments to D76077: [ARM] Add __bf16 as new Bfloat16 C Type.
May 14 2020, 8:38 AM · Restricted Project

May 13 2020

majnemer accepted D79474: [Windows EH] Fix the order of Nested try-catches in $tryMap$ table.

LGTM

May 13 2020, 1:03 PM · Restricted Project

May 6 2020

majnemer added inline comments to D79474: [Windows EH] Fix the order of Nested try-catches in $tryMap$ table.
May 6 2020, 1:00 PM · Restricted Project

Apr 13 2020

majnemer accepted D77897: [SVE] Remove calls to getBitWidth from IR.

LGTM

Apr 13 2020, 5:25 PM · Restricted Project
majnemer accepted D78047: [AntiDepBreaker][NFC] Move AntiDepBreaker.h to include folder.

LGTM

Apr 13 2020, 4:20 PM · Restricted Project

Apr 11 2020

majnemer added a comment to D77777: [nvptx] Add `nvvm.texsurf.handle` internalizer..
In D77777#1975406, @tra wrote:

the 1st argument in llvm.nvvm.texsurf.hande.internal or the 2nd one in llvm.nvvm.texsurf.handle must be kept as an immediate or constant value, i.e. that global variable. However, optimizations will find common code in the following

if (cond) {
  %hnd = texsurf.handle.internal(@tex1);
} else {
  %hnd = texsurf.handle.internal(@tex2)
}
= use(%hnd)

and hoist or sink it into

if (cond) {
  %ptr = @tex1;
} else {
  %ptr = @tex2;
}
%hnd = texsurf.handle.intenal(%ptr);
= use(%hnd)

The backend cannot handle non immediate operand in texsurf.handle. The similar thing happens to read.register as well as it also assumes its argument is always an immediate value.

I wonder if we can use token types to represent the handle? https://reviews.llvm.org/D11861
@majnemer -- would this use case be suitable for the token type?

If we still could make PHI over token, it canont serve this purpose. Check llvm::canReplaceOperandWithVariable for operand for details.

Apr 11 2020, 11:43 AM · Restricted Project

Apr 7 2020

majnemer added inline comments to D77437: [lld][WebAssembly] Handle 4gb max memories.
Apr 7 2020, 7:38 PM · Restricted Project

Mar 26 2020

majnemer accepted D76664: [ConstantFold][NFC] Compile time optimization for large vectors.

LGTM

Mar 26 2020, 10:50 AM · Restricted Project

Mar 24 2020

majnemer added inline comments to D76664: [ConstantFold][NFC] Compile time optimization for large vectors.
Mar 24 2020, 6:03 PM · Restricted Project
majnemer added inline comments to D76664: [ConstantFold][NFC] Compile time optimization for large vectors.
Mar 24 2020, 3:06 PM · Restricted Project
majnemer added a comment to D76664: [ConstantFold][NFC] Compile time optimization for large vectors.

Please upload with full context.

Mar 24 2020, 10:44 AM · Restricted Project

Mar 23 2020

majnemer added inline comments to D76664: [ConstantFold][NFC] Compile time optimization for large vectors.
Mar 23 2020, 7:36 PM · Restricted Project

Nov 3 2019

majnemer added a reviewer for D69776: [APFloat] Fix FP remainder operation: scanon.
Nov 3 2019, 1:42 PM · Restricted Project

Oct 28 2019

majnemer retitled D69477: [InstCombine] keep assumption before sinking calls from [InstCombine] keep assumption before skinking calls to [InstCombine] keep assumption before sinking calls.
Oct 28 2019, 3:45 PM · Restricted Project

Oct 17 2019

majnemer added inline comments to D69110: [DFAPacketizer] Allow up to 64 functional units.
Oct 17 2019, 10:33 AM · Restricted Project

Oct 8 2019

majnemer accepted D68570: Unify the two CRC implementations.

LGTM

Oct 8 2019, 10:36 AM · Restricted Project, Restricted Project, Restricted Project

Oct 2 2019

majnemer accepted D67968: [TableGen] Introduce a generic automaton (DFA) backend.

LGTM

Oct 2 2019, 4:20 PM · Restricted Project

Sep 26 2019

majnemer added inline comments to D67968: [TableGen] Introduce a generic automaton (DFA) backend.
Sep 26 2019, 2:33 PM · Restricted Project

Sep 19 2019

majnemer accepted D67356: [InstCombine] Simplify @llvm.usub.with.overflow+non-zero check (PR43251).

LGTM

Sep 19 2019, 10:01 AM · Restricted Project

Sep 12 2019

majnemer added inline comments to D67507: Refer to IEEE 754-2019 in langref instead of 2018 draft.
Sep 12 2019, 10:21 AM · Restricted Project

Sep 4 2019

majnemer added inline comments to D66827: Add support for MS qualifiers __ptr32, __ptr64, __sptr, __uptr..
Sep 4 2019, 3:29 PM · Restricted Project, Restricted Project

Aug 28 2019

majnemer added inline comments to D66836: [libc++] Add `__truncating_cast` for safely casting float types to integers.
Aug 28 2019, 5:35 PM · Restricted Project

Aug 23 2019

majnemer added a reviewer for D64665: [MachinePipeliner] Refactor schedule emission logic: ThomasRaoux.
Aug 23 2019, 2:23 PM · Restricted Project
majnemer added a reviewer for D66368: [MachinePipeliner] Factor experimental codegen into a class: ThomasRaoux.
Aug 23 2019, 2:15 PM · Restricted Project

Aug 14 2019

majnemer added inline comments to D66190: [CodeGen] Add a pass to do block predication on SSA machine IR.
Aug 14 2019, 5:12 PM · Restricted Project
majnemer added inline comments to D64665: [MachinePipeliner] Refactor schedule emission logic.
Aug 14 2019, 4:58 PM · Restricted Project

Jul 15 2019

majnemer added inline comments to D63396: [WinEH] Allocate space in funclets stack to save XMM CSRs.
Jul 15 2019, 9:39 PM · Restricted Project

Jul 6 2019

majnemer committed rG617df204b5b4: [CodeGen] Add larger vector types for i32 and f32 (authored by majnemer).
[CodeGen] Add larger vector types for i32 and f32
Jul 6 2019, 9:51 PM
majnemer committed rL365274: [CodeGen] Add larger vector types for i32 and f32.
[CodeGen] Add larger vector types for i32 and f32
Jul 6 2019, 9:48 PM
majnemer closed D64141: [CodeGen] Add larger vector types for i32 and f32.
Jul 6 2019, 9:48 PM · Restricted Project

Jul 3 2019

majnemer accepted D64141: [CodeGen] Add larger vector types for i32 and f32.

Most of this looks mechanical; I don't think the MVT changes should be contentious. However, the "is65536BitVector()" predicates are starting to get a bit unwieldy to read.

Does an out of tree target need these predicates? I feel they're sufficiently unreadable to either change the format ("is64kBitVector"?) or elide them for larger types.

Cheers,

James

I agree, it is not really needed I added it for consistency. I don't mind changing it to the format suggested. Which ones do you think should be changed?

Jul 3 2019, 11:39 AM · Restricted Project

Apr 1 2019

majnemer accepted D60079: [PruneEH] Don't split musttail call from ret.

LGTM

Apr 1 2019, 7:13 PM · Restricted Project
majnemer accepted D60080: [SimplifyCFG] Don't split musttail call from ret.

LGTM

Apr 1 2019, 7:13 PM · Restricted Project

Mar 12 2019

majnemer added inline comments to D59286: Allow pointer to static member function.
Mar 12 2019, 8:39 PM

Jan 23 2019

majnemer added a comment to D53765: [RFC prototype] Implementation of asm-goto support in LLVM.

Do you have any tests which perform a callbr with a normal function target rather than inline asm?

No, and Verifier.cpp currently errors for it.

Jan 23 2019, 3:49 PM · Restricted Project

Jan 22 2019

majnemer added a comment to D53765: [RFC prototype] Implementation of asm-goto support in LLVM.

Do you have any tests which perform a callbr with a normal function target rather than inline asm?

Jan 22 2019, 6:14 PM · Restricted Project

Jan 8 2019

majnemer added inline comments to D56466: [CodeGen] Clarify comment about COFF common symbol alignment.
Jan 8 2019, 4:51 PM

Jan 7 2019

majnemer added inline comments to D56391: Limit COFF 'common' emission to <=32 alignment types..
Jan 7 2019, 12:41 PM

Dec 18 2018

majnemer added inline comments to D55853: Ignore ConstantExpr in IgnoreParens.
Dec 18 2018, 2:17 PM

Dec 14 2018

majnemer added inline comments to D55715: Add AddressSpace mangling to MS mode.
Dec 14 2018, 11:44 AM

Nov 29 2018

majnemer accepted D55077: [Mem2Reg] Fix nondeterministic corner case.

LGTM

Nov 29 2018, 4:54 PM
majnemer accepted D54965: [obj2yaml] [COFF] Write RVA instead of VA for sections, fix roundtripping executables.

LGTM

Nov 29 2018, 10:45 AM

Nov 12 2018

majnemer added inline comments to D53877: [IR] Strawman for dedicated FNeg IR instruction.
Nov 12 2018, 8:37 AM · Restricted Project

Oct 31 2018

majnemer added inline comments to D53540: [COFF, ARM64] Implement support for SEH extensions __try/__except/__finally.
Oct 31 2018, 1:06 PM

Oct 8 2018

majnemer added inline comments to D53000: [Support] exit with custom return code for SIGPIPE.
Oct 8 2018, 3:59 PM · Restricted Project

Sep 27 2018

majnemer added inline comments to D52384: [Sema] Fix redeclaration contexts for enumerators in C.
Sep 27 2018, 10:53 PM

Sep 25 2018

majnemer accepted D52499: [clang-cl] Make /Gs imply default stack probes, not /Gs0 (PR39074).

FWIW, Microsoft's newest documentation does not say that /O2 and /O1 imply /Gs: https://docs.microsoft.com/en-us/cpp/build/reference/o1-o2-minimize-size-maximize-speed?view=vs-2017

Sep 25 2018, 11:19 AM

Sep 24 2018

majnemer added inline comments to D51524: [ARM64] [Windows] Handle funclets.
Sep 24 2018, 11:50 PM
majnemer added inline comments to D51524: [ARM64] [Windows] Handle funclets.
Sep 24 2018, 8:04 PM
majnemer added inline comments to D51524: [ARM64] [Windows] Handle funclets.
Sep 24 2018, 5:12 PM

Sep 7 2018

majnemer accepted D51820: [COFF] Implement llvm.global_ctors priorities for MSVC COFF targets.

LGTM

Sep 7 2018, 3:49 PM

Aug 22 2018

majnemer accepted D51149: MC: Don't align COFF section contents..

LGTM

Aug 22 2018, 10:10 PM

Aug 16 2018

majnemer added a comment to D50877: [MS] Mangle a hash of the main file path into anonymous namespaces.

How does MSVC handle this case? What mangled name does it generate?

Aug 16 2018, 5:27 PM

Aug 15 2018

majnemer accepted D50806: [MS Demangler] Demangle string literals.

LGTM

Aug 15 2018, 2:25 PM

Aug 7 2018

majnemer added inline comments to D50394: [MS Demangler] Properly handle backreferences of special names.
Aug 7 2018, 10:33 AM
majnemer accepted D50394: [MS Demangler] Properly handle backreferences of special names.

LGTM

Aug 7 2018, 10:33 AM

Aug 5 2018

majnemer added inline comments to D50222: [CodeGen] [SelectionDAG] More efficient code for X % C == 0 (UREM case).
Aug 5 2018, 2:39 PM · Restricted Project

Aug 4 2018

majnemer added inline comments to D50222: [CodeGen] [SelectionDAG] More efficient code for X % C == 0 (UREM case).
Aug 4 2018, 10:35 PM · Restricted Project

Aug 1 2018

majnemer accepted D50145: [MS Demangler] Demangle templated operator overloads.

LGTM

Aug 1 2018, 11:18 AM

Jul 29 2018

majnemer added inline comments to D49965: [MS Demangler] Demangle symbols in function local scopes.
Jul 29 2018, 7:32 PM
majnemer added a comment to D49965: [MS Demangler] Demangle symbols in function local scopes.

I had thought of a tryDemangleNumber function but it didn’t really fit the
style of code . (Ie I don’t really do that anywhere else so it felt a
little out of place). I can still do that if you think it’s better, I don’t
have a strong preference

Jul 29 2018, 6:09 PM
majnemer accepted D49965: [MS Demangler] Demangle symbols in function local scopes.
Jul 29 2018, 6:09 PM
majnemer added inline comments to D49965: [MS Demangler] Demangle symbols in function local scopes.
Jul 29 2018, 2:33 PM

Jul 25 2018

majnemer added inline comments to D49821: [MC] Add support for the .rva assembler directive for COFF targets.
Jul 25 2018, 2:50 PM

Jul 23 2018

majnemer added inline comments to D49644: [COFF] Hoist constant pool handling from X86AsmPrinter into AsmPrinter.
Jul 23 2018, 12:29 PM

Jul 22 2018

majnemer accepted D49644: [COFF] Hoist constant pool handling from X86AsmPrinter into AsmPrinter.

LGTM

Jul 22 2018, 12:24 PM

Jul 21 2018

majnemer added inline comments to D49630: [MS Demangler] Demangle data member pointers.
Jul 21 2018, 10:49 PM

Jul 19 2018

majnemer accepted D49552: Add a Microsoft Demangler library and utility..

I'm worried about some extreme template stuff (function templates are not considered for back-referencing but other templates and normal functions are OK) but you are gonna work thru the rest of mangle-ms-* so you'll get there ;)

Jul 19 2018, 8:21 PM
majnemer added inline comments to D49552: Add a Microsoft Demangler library and utility..
Jul 19 2018, 9:30 AM

Jul 15 2018

majnemer accepted D49354: [MinGW] Automatically mangle Windows-specific entry points as C.

LGTM

Jul 15 2018, 5:03 PM

Jul 3 2018

majnemer added inline comments to D48841: Add support for __declspec(code_seg("segname")).
Jul 3 2018, 3:46 PM

Jun 8 2018

majnemer accepted D47875: [MS ABI] Mangle unnamed empty enums (PR37723).

LGTM

Jun 8 2018, 4:36 PM

Jun 7 2018

majnemer added inline comments to D47875: [MS ABI] Mangle unnamed empty enums (PR37723).
Jun 7 2018, 7:54 AM

May 31 2018

majnemer accepted D47611: Change ambiguous uses of term 'funclet' to 'EH scopes'. NFC..

LGTM

May 31 2018, 4:59 PM

May 22 2018

majnemer added inline comments to D44134: [WebAssembly] Add WebAssemblyException information analysis.
May 22 2018, 8:49 AM

May 21 2018

majnemer added inline comments to D44134: [WebAssembly] Add WebAssemblyException information analysis.
May 21 2018, 8:39 AM
majnemer accepted D47005: [WebAssembly] Add functions for EHScopes.

LGTM

May 21 2018, 8:35 AM

May 20 2018

majnemer added inline comments to D44134: [WebAssembly] Add WebAssemblyException information analysis.
May 20 2018, 10:38 PM
majnemer added inline comments to D44134: [WebAssembly] Add WebAssemblyException information analysis.
May 20 2018, 10:22 PM
majnemer added inline comments to D44134: [WebAssembly] Add WebAssemblyException information analysis.
May 20 2018, 12:00 PM
majnemer added inline comments to D44931: [WebAssembly] Use Windows EH instructions for Wasm EH.
May 20 2018, 11:50 AM

May 18 2018

majnemer added inline comments to D43746: [WebAssembly] Add Wasm exception handling prepare pass.
May 18 2018, 8:45 AM

May 17 2018

majnemer added inline comments to D43746: [WebAssembly] Add Wasm exception handling prepare pass.
May 17 2018, 4:36 PM
majnemer accepted D47005: [WebAssembly] Add functions for EHScopes.

LGTM

May 17 2018, 4:07 PM