search for: desaulniers

...where the function `foo` gets redefined, and the JIT returns evaluation of the latest definition. I thought my code was wrong, but it seems that the binary produced by `ninja Kaleidoscope-Ch4` has the same bug. Granted, my LLVM checkout is from Nov 3 2018 (r346062). Is this a known issue? ~Nick Desaulniers

Similar to commit 0bf8bf50eddc ("module: Remove const attribute from alias for MODULE_DEVICE_TABLE") Fixes many -Wduplicate-decl-specifier warnings due to the combination of const typeof() of already const variables. Signed-off-by: Nick Desaulniers <ndesaulniers at google.com> --- drivers/gpu/drm/nouveau/include/nvif/object.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/include/nvif/object.h b/drivers/gpu/drm/nouveau/include/nvif/object.h index a2d5244ff2b7..7f188f66931e 100644 --- a/drive...

...ng who does JIT things > > Yeah, I can confirm my local build (using LLVM source from today) of Chapter 4 behaves as you describe, and not as the documentation shows. Looks like somethnig needs updating (either source or the documentation). > >> On Sun, Dec 30, 2018 at 3:28 PM Nick Desaulniers via llvm-dev <llvm-dev at lists.llvm.org> wrote: >> https://llvm.org/docs/tutorial/LangImpl04.html has an example where >> the function `foo` gets redefined, and the JIT returns evaluation of >> the latest definition. I thought my code was wrong, but it seems that >>...

* Nick Desaulniers <ndesaulniers at google.com> wrote: > On Thu, Jun 14, 2018 at 5:17 PM H. Peter Anvin <hpa at zytor.com> wrote: > > > > On 06/14/18 13:59, Nick Desaulniers wrote: > > > On Thu, Jun 14, 2018 at 1:48 PM H. Peter Anvin <hpa at zytor.com> wrote: > > >&...

...gt; >> Yeah, I can confirm my local build (using LLVM source from today) of >> Chapter 4 behaves as you describe, and not as the documentation shows. >> Looks like somethnig needs updating (either source or the documentation). >> >> On Sun, Dec 30, 2018 at 3:28 PM Nick Desaulniers via llvm-dev < >> llvm-dev at lists.llvm.org> wrote: >> >>> https://llvm.org/docs/tutorial/LangImpl04.html has an example where >>> the function `foo` gets redefined, and the JIT returns evaluation of >>> the latest definition. I thought my code was wro...

...gle.com_bbf8m6m4n8nq5p2bfjpele0n5s%40group.calendar.google.com [1] https://meet.google.com/yjf-jyqk-iaz [2] https://www.youtube.com/watch?v=UBghs-cwQX4 [3] https://hal.inria.fr/hal-02147012v2/document [4] https://hal.inria.fr/hal-02314830/document [5] https://www.mathieuacher.com/ -- Thanks, ~Nick Desaulniers

...gt; > Link: https://github.com/ClangBuiltLinux/linux/issues/982 > > Reported-by: Sedat Dilek <sedat.dilek at gmail.com> > > Signed-off-by: Nathan Chancellor <natechancellor at gmail.com> Thanks for the patch, sorry I'm falling behind on code review! Reviewed-by: Nick Desaulniers <ndesaulniers at google.com> > > --- > > arch/x86/mm/mmio-mod.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/arch/x86/mm/mmio-mod.c b/arch/x86/mm/mmio-mod.c > > index 109325d77b3e..43fd19b3f118 100644 > > --- a/...

...h, maybe Clang should continue to warn for the above (at least the `default: return;` case) and GCC should change? While the last case looks harmless, there were only 1 or 2 across the tree in my limited configuration testing; I really think we should just add `break`s for those. -- Thanks, ~Nick Desaulniers

...h, maybe Clang should continue to warn for the above (at least the `default: return;` case) and GCC should change? While the last case looks harmless, there were only 1 or 2 across the tree in my limited configuration testing; I really think we should just add `break`s for those. -- Thanks, ~Nick Desaulniers

...h, maybe Clang should continue to warn for the above (at least the `default: return;` case) and GCC should change? While the last case looks harmless, there were only 1 or 2 across the tree in my limited configuration testing; I really think we should just add `break`s for those. -- Thanks, ~Nick Desaulniers

...sn't seem to be worth it. ``` Looks like there's an order of magnitude of `-Wunreachable-code-break` than the other two. We probably should add all 3 to W=2 builds (wrapped in cc-option). I've filed https://github.com/ClangBuiltLinux/linux/issues/1180 to follow up on. -- Thanks, ~Nick Desaulniers

On Fri, Jun 15, 2018 at 3:56 PM Nick Desaulniers <ndesaulniers at google.com> wrote: > > Similar to commit 0bf8bf50eddc ("module: Remove > const attribute from alias for MODULE_DEVICE_TABLE") > > Fixes many -Wduplicate-decl-specifier warnings due to the combination of > const typeof() of already const variables....

Similar to commit 0bf8bf50eddc ("module: Remove const attribute from alias for MODULE_DEVICE_TABLE") Fixes many -Wduplicate-decl-specifier warnings due to the combination of const typeof() of already const variables. Signed-off-by: Nick Desaulniers <ndesaulniers at google.com> --- Changes since v1: added additional space after statements. drivers/gpu/drm/nouveau/include/nvif/object.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/include/nvif/object.h b/drivers/gpu/drm/nouveau/include/nvif/...

On 06/05/18 10:05, Nick Desaulniers wrote: > + > +/* > + * void native_restore_fl(unsigned long flags) > + * %rdi: flags > + */ > +ENTRY(native_restore_fl) > + push %_ASM_DI > + popf > + ret > +ENDPROC(native_restore_fl) > +EXPORT_SYMBOL(native_restore_fl) > To work on i386, this would have to be...

On 06/14/18 13:59, Nick Desaulniers wrote: > On Thu, Jun 14, 2018 at 1:48 PM H. Peter Anvin <hpa at zytor.com> wrote: >> >> On 06/13/18 14:05, Nick Desaulniers wrote: >>> From: "H. Peter Anvin" <hpa at linux.intel.com> >>> >>> i386 and x86-64 uses different registers for...

On 06/05/18 10:05, Nick Desaulniers wrote: > + > +/* > + * void native_restore_fl(unsigned long flags) > + * %rdi: flags > + */ > +ENTRY(native_restore_fl) > + push %_ASM_DI > + popf > + ret > +ENDPROC(native_restore_fl) > +EXPORT_SYMBOL(native_restore_fl) > To work on i386, this would have to be...

[Adding the correct cfe-dev mailing list address.] On Thu, Jun 27, 2019 at 11:06 AM Bill Wendling <isanbard at gmail.com> wrote: > Now that ASM goto support has landed, Nick Desaulniers and I wrote up a > document describing how to expand clang's implementation of ASM goto to > support output constraints. The work *should* be straight-forward, but as > always will need to be verified to work. Below is a copy of our whitepaper. > Please take a look and offer any com...

....com>,Matthias Kaehlcke <mka at chromium.org>,thomas.lendacky at amd.com,Thiebaud Weksteen <tweek at google.com>,mjg59 at google.com,joe at perches.com From: hpa at zytor.com Message-ID: <191E4EBE-4CB2-4C8B-AB61-689A91FFE7A8 at zytor.com> On June 12, 2018 11:33:14 AM PDT, Nick Desaulniers <ndesaulniers at google.com> wrote: >On Fri, Jun 8, 2018 at 3:04 AM Sedat Dilek <sedat.dilek at gmail.com> >wrote: >> >> On Fri, Jun 8, 2018 at 9:59 AM, Arnd Bergmann <arnd at arndb.de> wrote: >> > On Thu, Jun 7, 2018 at 10:49 PM, Nick Desaulniers >&g...

What about SelectionDAG representation? Currently we expand callbr to INLINEASM_BR and BR. Both of which are terminators. But in order to support outputs we would need to put CopyFromReg nodes between them. ~Craig On Thu, Jun 27, 2019 at 12:18 PM Nick Desaulniers via cfe-dev < cfe-dev at lists.llvm.org> wrote: > + CBL mailing list > > > On Thu, Jun 27, 2019 at 11:08 AM Bill Wendling <isanbard at gmail.com> wrote: > >> [Adding the correct cfe-dev mailing list address.] >> >> On Thu, Jun 27, 2019 at 11:06 AM Bill...

...-inline-declarations.patch and it can be found in the queue-4.4 subdirectory. If you, or anyone else, feels it should not be added to the stable tree, please let <stable at vger.kernel.org> know about it. >From d03db2bc26f0e4a6849ad649a09c9c73fccdc656 Mon Sep 17 00:00:00 2001 From: Nick Desaulniers <ndesaulniers at google.com> Date: Thu, 21 Jun 2018 09:23:22 -0700 Subject: compiler-gcc.h: Add __attribute__((gnu_inline)) to all inline declarations From: Nick Desaulniers <ndesaulniers at google.com> commit d03db2bc26f0e4a6849ad649a09c9c73fccdc656 upstream. Functions marked extern...