Page MenuHomePhabricator
Feed Advanced Search

Today

nickdesaulniers added inline comments to D70062: MCObjectStreamer: assign MCSymbols in the dummy fragment to offset 0..
Mon, Nov 11, 10:59 AM · Restricted Project
nickdesaulniers committed rG49fb4a96e0b7: change LLVM_VERSION_SUFFIX default from svn to git (authored by nickdesaulniers).
change LLVM_VERSION_SUFFIX default from svn to git
Mon, Nov 11, 9:17 AM
nickdesaulniers closed D70019: change LLVM_VERSION_SUFFIX default from svn to git.
Mon, Nov 11, 9:17 AM · Restricted Project, Restricted Project

Fri, Nov 8

nickdesaulniers created D70019: change LLVM_VERSION_SUFFIX default from svn to git.
Fri, Nov 8, 10:35 AM · Restricted Project, Restricted Project
nickdesaulniers added a reviewer for D70019: change LLVM_VERSION_SUFFIX default from svn to git: hans.
Fri, Nov 8, 10:35 AM · Restricted Project, Restricted Project
nickdesaulniers committed rG51adeae1c90c: remove redundant LLVM version from version string when setting CLANG_VENDOR (authored by nickdesaulniers).
remove redundant LLVM version from version string when setting CLANG_VENDOR
Fri, Nov 8, 9:31 AM
nickdesaulniers closed D69925: remove redundant LLVM version from version string when setting CLANG_VENDOR.
Fri, Nov 8, 9:31 AM · Restricted Project

Thu, Nov 7

nickdesaulniers added inline comments to D69868: Allow "callbr" to return non-void values.
Thu, Nov 7, 2:45 PM · Restricted Project
nickdesaulniers added a comment to D69925: remove redundant LLVM version from version string when setting CLANG_VENDOR.

Thanks for the review, leaving open for final comments, will merge tomorrow.

Thu, Nov 7, 2:45 PM · Restricted Project
nickdesaulniers added a comment to D69925: remove redundant LLVM version from version string when setting CLANG_VENDOR.

Just grabbed a version string off a MBP:

Thu, Nov 7, 12:50 PM · Restricted Project
nickdesaulniers added a comment to D69925: remove redundant LLVM version from version string when setting CLANG_VENDOR.

+1, Apple's clang has an #ifdef 0 around this code.

Thu, Nov 7, 12:32 PM · Restricted Project

Wed, Nov 6

nickdesaulniers added inline comments to D69925: remove redundant LLVM version from version string when setting CLANG_VENDOR.
Wed, Nov 6, 3:42 PM · Restricted Project
nickdesaulniers created D69925: remove redundant LLVM version from version string when setting CLANG_VENDOR.
Wed, Nov 6, 3:42 PM · Restricted Project
nickdesaulniers added inline comments to D69925: remove redundant LLVM version from version string when setting CLANG_VENDOR.
Wed, Nov 6, 3:42 PM · Restricted Project
nickdesaulniers added inline comments to D69870: [builtins] Implement rounding mode support for i386/x86_64.
Wed, Nov 6, 10:29 AM · Restricted Project, Restricted Project
nickdesaulniers added inline comments to D69411: [MC] Parse .if conditions with symbols in consecutive MCDataFragements.
Wed, Nov 6, 10:09 AM · Restricted Project
nickdesaulniers added inline comments to D69411: [MC] Parse .if conditions with symbols in consecutive MCDataFragements.
Wed, Nov 6, 10:09 AM · Restricted Project
nickdesaulniers added a reviewer for D69870: [builtins] Implement rounding mode support for i386/x86_64: nickdesaulniers.
Wed, Nov 6, 9:59 AM · Restricted Project, Restricted Project
nickdesaulniers added inline comments to D69870: [builtins] Implement rounding mode support for i386/x86_64.
Wed, Nov 6, 9:59 AM · Restricted Project, Restricted Project
nickdesaulniers edited reviewers for D69411: [MC] Parse .if conditions with symbols in consecutive MCDataFragements, added: peter.smith, MaskRay; removed: dschuff, sunfish.
Wed, Nov 6, 9:31 AM · Restricted Project

Tue, Nov 5

nickdesaulniers updated subscribers of D69411: [MC] Parse .if conditions with symbols in consecutive MCDataFragements.
Tue, Nov 5, 12:01 PM · Restricted Project

Wed, Oct 30

nickdesaulniers added a comment to D69296: [ARM] Uses "Sun Style" syntax for section switching.

Linux has been patched by to use normal section switching syntax https://lkml.org/lkml/2019/10/30/807. Abandon this LLVM patch as we should be fine without it.

Wed, Oct 30, 11:31 AM · Restricted Project

Tue, Oct 29

nickdesaulniers added a comment to D69296: [ARM] Uses "Sun Style" syntax for section switching.

The original patch looks fine because the SunStyleELFSectionSwitchSyntax functionality is already there. Now we start to add IsCompatibleWithSunStyleELFSectionSwitchSyntax... Have we communicated the Solaris assembler's .section syntax problem to the Linux community?

Tue, Oct 29, 2:17 PM · Restricted Project
nickdesaulniers added a comment to D69411: [MC] Parse .if conditions with symbols in consecutive MCDataFragements.

There are also some test cases for other ISA's in https://bugs.llvm.org/show_bug.cgi?id=41825.

Tue, Oct 29, 9:09 AM · Restricted Project

Mon, Oct 28

nickdesaulniers added a reviewer for D69411: [MC] Parse .if conditions with symbols in consecutive MCDataFragements: nickdesaulniers.
Mon, Oct 28, 12:35 PM · Restricted Project
nickdesaulniers added a comment to D69411: [MC] Parse .if conditions with symbols in consecutive MCDataFragements.

Thanks for the patch! Looks like this was much simpler to support than I expected.

Mon, Oct 28, 12:21 PM · Restricted Project

Fri, Oct 25

nickdesaulniers added inline comments to D69296: [ARM] Uses "Sun Style" syntax for section switching.
Fri, Oct 25, 10:06 AM · Restricted Project

Wed, Oct 16

nickdesaulniers added a comment to D69018: [AArch64] Fix offset calculation.

Likely also fixes regressions for aarch64 linux kernel builds: https://travis-ci.com/ClangBuiltLinux/continuous-integration/jobs/246197698

Wed, Oct 16, 4:23 PM · Restricted Project
nickdesaulniers added inline comments to rL369601: [TSan] #include header instead of forward declaring interceptees.
Wed, Oct 16, 1:52 PM

Mon, Oct 14

nickdesaulniers accepted D68764: [ARM][AsmParser] handles offset expression in parentheses.

Thanks for the patch and for following up on code review!

Mon, Oct 14, 2:21 PM · Restricted Project

Oct 11 2019

nickdesaulniers added inline comments to D68764: [ARM][AsmParser] handles offset expression in parentheses.
Oct 11 2019, 2:04 PM · Restricted Project
nickdesaulniers updated the summary of D68884: Add support to -Wa,-W in clang.
Oct 11 2019, 1:24 PM · Restricted Project
nickdesaulniers added inline comments to D68764: [ARM][AsmParser] handles offset expression in parentheses.
Oct 11 2019, 9:10 AM · Restricted Project

Oct 10 2019

nickdesaulniers added inline comments to D68764: [ARM][AsmParser] handles offset expression in parentheses.
Oct 10 2019, 1:16 PM · Restricted Project
nickdesaulniers added a comment to D68764: [ARM][AsmParser] handles offset expression in parentheses.

Great test cases. Thanks for the patch!

Oct 10 2019, 1:07 PM · Restricted Project
nickdesaulniers added a comment to D67986: [InstCombine] snprintf (d, size, "%s", s) -> memccpy (d, s, '\0', size - 1), d[size - 1] = 0.

This transformation seems to increase code size significantly. Is the snprintf "%s" pattern common enough? I suspect most projects have already used memccpy, stpncpy, strscpy, or strlcpy. For the few that don't, the performance probably does not matter.

Oct 10 2019, 12:59 PM · Restricted Project

Oct 7 2019

nickdesaulniers added a comment to D68410: [AttrDocs] document always_inline.

Also, I'm fairly certain __forceinline and always_inline, confusingly enough differ in semantics, with __forceinline only being a stronger hint on MSVC.

Oct 7 2019, 11:26 AM · Restricted Project

Oct 4 2019

nickdesaulniers added a reviewer for D68483: [IA] Recognize hexadecimal escape sequences: jcai19.
Oct 4 2019, 2:30 PM · Restricted Project

Oct 3 2019

nickdesaulniers added a reviewer for D67945: LowerTypeTests: Rename local functions to avoid collisions with identically named functions in ThinLTO modules.: dexonsmith.
Oct 3 2019, 3:38 PM · Restricted Project
nickdesaulniers added inline comments to D68410: [AttrDocs] document always_inline.
Oct 3 2019, 3:29 PM · Restricted Project
nickdesaulniers added reviewers for D68410: [AttrDocs] document always_inline: jyknight, lebedev.ri, ojeda, jdoerfert, erichkeane, dblaikie, kristina.
Oct 3 2019, 2:08 PM · Restricted Project
nickdesaulniers updated the diff for D68410: [AttrDocs] document always_inline.
  • add links
  • remove extra whitespace
  • rewording, split onto separate lines to ease code review
  • add spellings to heading
Oct 3 2019, 2:05 PM · Restricted Project
nickdesaulniers committed rGede784ff5a60: [AArch64InstPrinter] prefer bfi to bfc for < armv8.2-a (authored by nickdesaulniers).
[AArch64InstPrinter] prefer bfi to bfc for < armv8.2-a
Oct 3 2019, 1:11 PM
nickdesaulniers committed rL373655: [AArch64InstPrinter] prefer bfi to bfc for < armv8.2-a.
[AArch64InstPrinter] prefer bfi to bfc for < armv8.2-a
Oct 3 2019, 1:11 PM
nickdesaulniers closed D68356: [AArch64InstPrinter] prefer bfi to bfc for < armv8.2-a.
Oct 3 2019, 1:11 PM · Restricted Project
nickdesaulniers updated the summary of D68410: [AttrDocs] document always_inline.
Oct 3 2019, 12:56 PM · Restricted Project
nickdesaulniers created D68410: [AttrDocs] document always_inline.
Oct 3 2019, 11:37 AM · Restricted Project
nickdesaulniers updated the diff for D68356: [AArch64InstPrinter] prefer bfi to bfc for < armv8.2-a.
  • fix two unit tests that needed updating
Oct 3 2019, 10:01 AM · Restricted Project
nickdesaulniers added a comment to D68356: [AArch64InstPrinter] prefer bfi to bfc for < armv8.2-a.

Sorry, this breaks:

Oct 3 2019, 9:46 AM · Restricted Project
nickdesaulniers updated the summary of D68356: [AArch64InstPrinter] prefer bfi to bfc for < armv8.2-a.
Oct 3 2019, 9:00 AM · Restricted Project

Oct 2 2019

nickdesaulniers updated the diff for D68356: [AArch64InstPrinter] prefer bfi to bfc for < armv8.2-a.
  • git-clang-format HEAD~
Oct 2 2019, 1:39 PM · Restricted Project
nickdesaulniers created D68356: [AArch64InstPrinter] prefer bfi to bfc for < armv8.2-a.
Oct 2 2019, 1:39 PM · Restricted Project
nickdesaulniers added a comment to D67986: [InstCombine] snprintf (d, size, "%s", s) -> memccpy (d, s, '\0', size - 1), d[size - 1] = 0.

It sounds like memccpy is part of C20. Can we not do this transform unless LangOpt says we're C20 or greater? Otherwise the Linux kernel doesn't implement this routine.

Oct 2 2019, 10:27 AM · Restricted Project

Oct 1 2019

nickdesaulniers updated subscribers of D67469: [WIP][Debuginfo][LLD] Remove obsolete debug info while garbage collecting..
Oct 1 2019, 11:02 PM · debug-info, lld, Restricted Project
nickdesaulniers accepted D68055: Add -fgnuc-version= to control __GNUC__ and other GCC macros.
Oct 1 2019, 11:00 PM · Restricted Project
nickdesaulniers added inline comments to D68131: Switch lowering: omit range check for bit tests when default is unreachable (PR43129).
Oct 1 2019, 1:25 PM · Restricted Project
nickdesaulniers added inline comments to D68131: Switch lowering: omit range check for bit tests when default is unreachable (PR43129).
Oct 1 2019, 9:54 AM · Restricted Project

Sep 30 2019

nickdesaulniers added inline comments to D68131: Switch lowering: omit range check for bit tests when default is unreachable (PR43129).
Sep 30 2019, 2:33 PM · Restricted Project

Sep 27 2019

nickdesaulniers added a comment to D68131: Switch lowering: omit range check for bit tests when default is unreachable (PR43129).

Thanks for the patch! Just some small thoughts on style.

Sep 27 2019, 1:22 PM · Restricted Project

Sep 25 2019

nickdesaulniers committed rG93d87260f1f7: [Verifier] add invariant check for callbr (authored by nickdesaulniers).
[Verifier] add invariant check for callbr
Sep 25 2019, 3:28 PM
nickdesaulniers committed rL372923: [Verifier] add invariant check for callbr.
[Verifier] add invariant check for callbr
Sep 25 2019, 3:28 PM
nickdesaulniers closed D67196: [Verifier] add invariant check for callbr.
Sep 25 2019, 3:28 PM · Restricted Project
nickdesaulniers added a comment to D68050: WIP Make attribute target work better with AArch64.

Thanks for the patch. No comment if this is "the right thing to do," just basic code review comments.

Sep 25 2019, 3:01 PM · Restricted Project

Sep 23 2019

nickdesaulniers added a comment to D67196: [Verifier] add invariant check for callbr.

Bumping for code review

Sep 23 2019, 1:09 PM · Restricted Project
nickdesaulniers added a comment to D67873: Allow llvm-objdump to run on executable files.

Please post patches with context.

Sep 23 2019, 9:07 AM · Restricted Project

Sep 17 2019

nickdesaulniers added a reviewer for D67196: [Verifier] add invariant check for callbr: chandlerc.
Sep 17 2019, 10:35 PM · Restricted Project

Sep 12 2019

nickdesaulniers added inline comments to D67414: [AST] Treat "inline gnu_inline" the same way as "extern inline gnu_inline" in C++ mode.
Sep 12 2019, 1:38 PM · Restricted Project, Restricted Project
nickdesaulniers added reviewers for D67414: [AST] Treat "inline gnu_inline" the same way as "extern inline gnu_inline" in C++ mode: aaron.ballman, rsmith.
Sep 12 2019, 1:01 PM · Restricted Project, Restricted Project
nickdesaulniers committed rG5806022904bc: [Clang][CodeGen] support alias attribute w/ gnu_inline (authored by nickdesaulniers).
[Clang][CodeGen] support alias attribute w/ gnu_inline
Sep 12 2019, 12:54 PM
nickdesaulniers committed rL371766: [Clang][CodeGen] support alias attribute w/ gnu_inline.
[Clang][CodeGen] support alias attribute w/ gnu_inline
Sep 12 2019, 12:52 PM
nickdesaulniers closed D67455: [Clang][CodeGen] support alias attribute w/ gnu_inline.
Sep 12 2019, 12:52 PM · Restricted Project, Restricted Project

Sep 11 2019

nickdesaulniers updated the diff for D67455: [Clang][CodeGen] support alias attribute w/ gnu_inline.
  • adjust parens
Sep 11 2019, 1:49 PM · Restricted Project, Restricted Project
nickdesaulniers created D67455: [Clang][CodeGen] support alias attribute w/ gnu_inline.
Sep 11 2019, 11:48 AM · Restricted Project, Restricted Project

Sep 6 2019

nickdesaulniers committed rL371288: Fix ndesaulniers svn account, s/ndesaulniers/nickdesaulniers.
Fix ndesaulniers svn account, s/ndesaulniers/nickdesaulniers
Sep 6 2019, 10:09 PM
nickdesaulniers committed rL371287: Request commit access for ndesaulniers.
Request commit access for ndesaulniers
Sep 6 2019, 10:06 PM
nickdesaulniers updated subscribers of D67306: [IfConversion] Correctly handle cases where analyzeBranch fails..

FWIW, I verified that this patch fixes pr/42012 and the test case pases. Thanks @eli.friedman !

Sep 6 2019, 5:16 PM · Restricted Project
nickdesaulniers updated the diff for D67196: [Verifier] add invariant check for callbr.
  • rebase
Sep 6 2019, 3:18 PM · Restricted Project
nickdesaulniers committed rG7a7bba289521: [IR] CallBrInst: scan+update arg list when indirect dest list changes (authored by nickdesaulniers).
[IR] CallBrInst: scan+update arg list when indirect dest list changes
Sep 6 2019, 2:52 PM
nickdesaulniers committed rL371262: [IR] CallBrInst: scan+update arg list when indirect dest list changes.
[IR] CallBrInst: scan+update arg list when indirect dest list changes
Sep 6 2019, 2:51 PM
nickdesaulniers closed D67252: [IR] CallBrInst: scan+update arg list when indirect dest list changes.
Sep 6 2019, 2:51 PM · Restricted Project
nickdesaulniers updated the diff for D67252: [IR] CallBrInst: scan+update arg list when indirect dest list changes.
  • git-clang-format HEAD~
Sep 6 2019, 2:24 PM · Restricted Project
nickdesaulniers updated the summary of D67252: [IR] CallBrInst: scan+update arg list when indirect dest list changes.
Sep 6 2019, 12:30 PM · Restricted Project
nickdesaulniers updated the diff for D67252: [IR] CallBrInst: scan+update arg list when indirect dest list changes.
  • updated commit message
Sep 6 2019, 12:30 PM · Restricted Project
nickdesaulniers added a comment to D67252: [IR] CallBrInst: scan+update arg list when indirect dest list changes.

I have verified this allows me to boot:

  • arm64 linux-next (this patch fixes this regression, the below I verified are not regressed)
  • x86_64 linux-next
  • arm linux-next
  • arm linux-next + CONFIG_JUMP_LABEL
  • arm64 linux
  • x86_64 linux
  • arm linux
  • arm linux + CONFIG_JUMP_LABEL
Sep 6 2019, 12:10 PM · Restricted Project
nickdesaulniers added inline comments to D67253: clang-misexpect: a standalone tool for verifying the use of __builtin_expect with PGO data.
Sep 6 2019, 10:45 AM · Restricted Project, Restricted Project
nickdesaulniers updated the diff for D67252: [IR] CallBrInst: scan+update arg list when indirect dest list changes.
  • rebase, add/clean up the commit message
Sep 6 2019, 10:17 AM · Restricted Project
nickdesaulniers added inline comments to D67252: [IR] CallBrInst: scan+update arg list when indirect dest list changes.
Sep 6 2019, 10:14 AM · Restricted Project
nickdesaulniers abandoned D62555: [TailDuplicator] prevent tail duplication for INLINEASM_BR.

@eli.friedman fixed this in r371111. Thanks Eli!

Sep 6 2019, 10:00 AM · Restricted Project

Sep 5 2019

nickdesaulniers updated subscribers of D67252: [IR] CallBrInst: scan+update arg list when indirect dest list changes.
Sep 5 2019, 6:24 PM · Restricted Project
nickdesaulniers created D67252: [IR] CallBrInst: scan+update arg list when indirect dest list changes.
Sep 5 2019, 6:24 PM · Restricted Project

Sep 4 2019

nickdesaulniers updated the diff for D67196: [Verifier] add invariant check for callbr.
  • fix my added unit test
Sep 4 2019, 4:36 PM · Restricted Project
nickdesaulniers updated the diff for D67196: [Verifier] add invariant check for callbr.
  • add unit test, add invariant to LangRef, remove "or jump" from LangRef (was never there)
Sep 4 2019, 4:27 PM · Restricted Project
nickdesaulniers added a comment to D67196: [Verifier] add invariant check for callbr.

Is this already documented in LangRef?

Sep 4 2019, 3:41 PM · Restricted Project
nickdesaulniers updated the diff for D67196: [Verifier] add invariant check for callbr.
  • rebase on master, run git-clang-format HEAD~
Sep 4 2019, 3:35 PM · Restricted Project
nickdesaulniers created D67196: [Verifier] add invariant check for callbr.
Sep 4 2019, 3:27 PM · Restricted Project

Aug 28 2019

nickdesaulniers accepted D66324: clang-misexpect: Profile Guided Validation of Performance Annotations in LLVM.

Ok, thanks for the iteration on code review.

Aug 28 2019, 6:41 PM · Restricted Project, Restricted Project
nickdesaulniers added a comment to D66324: clang-misexpect: Profile Guided Validation of Performance Annotations in LLVM.

Looks good, will likely approve after these 2 questions.

Aug 28 2019, 4:36 PM · Restricted Project, Restricted Project

Aug 27 2019

nickdesaulniers added inline comments to D66324: clang-misexpect: Profile Guided Validation of Performance Annotations in LLVM.
Aug 27 2019, 4:01 PM · Restricted Project, Restricted Project
nickdesaulniers added inline comments to D66324: clang-misexpect: Profile Guided Validation of Performance Annotations in LLVM.
Aug 27 2019, 2:48 PM · Restricted Project, Restricted Project
nickdesaulniers added inline comments to D66324: clang-misexpect: Profile Guided Validation of Performance Annotations in LLVM.
Aug 27 2019, 1:34 PM · Restricted Project, Restricted Project