jgorbe (Jorge Gorbe Moya)
User

Projects

User does not belong to any projects.

User Details

User Since
Feb 9 2017, 3:56 PM (92 w, 15 h)

Recent Activity

Wed, Nov 14

jgorbe added inline comments to D54550: Mark lambda decl as invalid if a captured variable has an invalid type..
Wed, Nov 14, 4:27 PM
jgorbe updated the diff for D54550: Mark lambda decl as invalid if a captured variable has an invalid type..

Fixed some issues pointed out in review comments:

  • call to getBaseElementType before checking type validity.
  • when the type is incomplete, mark not only the lambda closure type as invalid but also the field
Wed, Nov 14, 4:25 PM
jgorbe created D54550: Mark lambda decl as invalid if a captured variable has an invalid type..
Wed, Nov 14, 3:26 PM

Thu, Nov 8

jgorbe committed rLLDB346441: [lldb] Silence unhandled enums warning..
[lldb] Silence unhandled enums warning.
Thu, Nov 8, 2:07 PM
jgorbe committed rL346441: [lldb] Silence unhandled enums warning..
[lldb] Silence unhandled enums warning.
Thu, Nov 8, 2:07 PM
jgorbe committed rLLDB346435: [lldb] Fix initialization order warnings..
[lldb] Fix initialization order warnings.
Thu, Nov 8, 12:01 PM
jgorbe committed rL346435: [lldb] Fix initialization order warnings..
[lldb] Fix initialization order warnings.
Thu, Nov 8, 12:01 PM

Wed, Nov 7

jgorbe committed rL346365: Add parentheses to silence warning..
Add parentheses to silence warning.
Wed, Nov 7, 2:32 PM

Sep 21 2018

jgorbe committed rL342771: Test commit. Removed a blank line..
Test commit. Removed a blank line.
Sep 21 2018, 1:46 PM

Sep 19 2018

jgorbe created D52280: Don't trim non-alphanumeric characters in 'file not found' errors for include directives..
Sep 19 2018, 2:34 PM

Aug 17 2018

jgorbe added a comment to D46179: [X86] Lowering addus/subus intrinsics to native IR (LLVM part).

I think this change is breaking one of our builds. The attached reduced test case fails with the current trunk revision if built with "clang -x c -O2 -mavx -c crash.ii".

Aug 17 2018, 3:58 AM

Apr 26 2018

jgorbe added a comment to D46135: [Driver, CodeGen] add options to enable/disable an FP cast optimization.

Can't comment much on the patch itself (I'm still not very familiar with the codebase, I'll leave that to the other reviewers), but thanks a lot for responding so quickly! :)

Apr 26 2018, 1:08 PM

Jun 5 2017

jgorbe added inline comments to D30963: Fix crash when an 'import a module' TypoCorrection has its CorrectionDecls replaced by visible decls..
Jun 5 2017, 11:56 AM
jgorbe updated the diff for D30963: Fix crash when an 'import a module' TypoCorrection has its CorrectionDecls replaced by visible decls..

Removed unneeded branch.

Jun 5 2017, 11:54 AM

Jun 2 2017

jgorbe added inline comments to D30963: Fix crash when an 'import a module' TypoCorrection has its CorrectionDecls replaced by visible decls..
Jun 2 2017, 6:07 PM
jgorbe updated the diff for D30963: Fix crash when an 'import a module' TypoCorrection has its CorrectionDecls replaced by visible decls..

Also clear the 'requires import' flag when the TypoCorrection has no decls at all.

Jun 2 2017, 6:03 PM
jgorbe added a comment to D30963: Fix crash when an 'import a module' TypoCorrection has its CorrectionDecls replaced by visible decls..

Ping?

Jun 2 2017, 5:04 PM

Jun 1 2017

jgorbe added inline comments to D33108: Generate extra .ll files before/after optimization when using -save-temps..
Jun 1 2017, 5:00 PM

May 31 2017

jgorbe added inline comments to D33108: Generate extra .ll files before/after optimization when using -save-temps..
May 31 2017, 6:15 PM
jgorbe updated the diff for D33108: Generate extra .ll files before/after optimization when using -save-temps..

Clarify that we only skip saving IR for multiarch Mach-O universal builds, not other multi-arch builds like CUDA.

May 31 2017, 6:15 PM
jgorbe added a reviewer for D33108: Generate extra .ll files before/after optimization when using -save-temps.: chandlerc.
May 31 2017, 5:12 PM
jgorbe updated the diff for D33108: Generate extra .ll files before/after optimization when using -save-temps..

Only generate extra .ll files when not targeting multiple archs. Updated related tests.

May 31 2017, 5:09 PM

May 11 2017

jgorbe updated the summary of D33108: Generate extra .ll files before/after optimization when using -save-temps..
May 11 2017, 4:00 PM
jgorbe created D33108: Generate extra .ll files before/after optimization when using -save-temps..
May 11 2017, 3:49 PM

Apr 26 2017

jgorbe created D32573: [Driver] Rename GetNamedOutputPath() to addNamedOutputPath().
Apr 26 2017, 5:19 PM

Mar 16 2017

jgorbe updated the diff for D30317: [X86] Clean up test/CodeGen/X86/2006-03-01-InstrSchedBug.ll.

Added -mtriple parameter to the RUN line. I messed up when regenerating the test with the script, reverted it, and lost the edit I had done before. Thanks for noticing.

Mar 16 2017, 4:30 PM
jgorbe updated the diff for D30317: [X86] Clean up test/CodeGen/X86/2006-03-01-InstrSchedBug.ll.
  • Regenerated the test with update_llc_test_checks.py
  • Specified target triple in the RUN: line with -mtriple
  • Removed entry label added in the previous version of the patch.
Mar 16 2017, 4:01 PM

Mar 14 2017

jgorbe created D30963: Fix crash when an 'import a module' TypoCorrection has its CorrectionDecls replaced by visible decls..
Mar 14 2017, 6:00 PM

Feb 23 2017

jgorbe created D30320: [X86] Clean up test/CodeGen/X86/2006-03-02-InstrSchedBug.ll.
Feb 23 2017, 6:47 PM
jgorbe created D30317: [X86] Clean up test/CodeGen/X86/2006-03-01-InstrSchedBug.ll.
Feb 23 2017, 5:22 PM

Feb 16 2017

jgorbe added inline comments to D29807: FileCheck-ize some tests in test/CodeGen/X86/.
Feb 16 2017, 4:19 PM

Feb 13 2017

jgorbe added a comment to D29807: FileCheck-ize some tests in test/CodeGen/X86/.

Some more answers to inline comments. Thanks!

Feb 13 2017, 11:59 AM
jgorbe updated the diff for D29807: FileCheck-ize some tests in test/CodeGen/X86/.
  • Added explicit target triple to test/CodeGen/X86/2004-02-14-InefficientStackPointer.ll
  • Added regex to catch possible constant displacements from %esp in test/CodeGen/X86/2004-02-13-FrameReturnAddress.ll
  • Added missing entry label.
Feb 13 2017, 11:59 AM

Feb 10 2017

jgorbe added a comment to D29807: FileCheck-ize some tests in test/CodeGen/X86/.

Some responses to @chandlerc 's previous comments.

Feb 10 2017, 7:20 PM
jgorbe updated the diff for D29807: FileCheck-ize some tests in test/CodeGen/X86/.

Made some changes suggested by chandlerc:

Feb 10 2017, 7:20 PM

Feb 9 2017

jgorbe added a comment to D29807: FileCheck-ize some tests in test/CodeGen/X86/.

Noted, thanks!

Feb 9 2017, 7:16 PM
jgorbe added a comment to D29807: FileCheck-ize some tests in test/CodeGen/X86/.

Forgot to add llvm-commits, commenting so a message gets posted there. Sorry!

Feb 9 2017, 7:10 PM
jgorbe updated subscribers of D29807: FileCheck-ize some tests in test/CodeGen/X86/.
Feb 9 2017, 7:08 PM
jgorbe created D29807: FileCheck-ize some tests in test/CodeGen/X86/.
Feb 9 2017, 7:08 PM
jgorbe added a comment to D29797: Update test/CodeGen/X86/sse-align-10.ll to use FileCheck instead of grep.

@dlj Can you please commit this patch? I don't have commit access.

Feb 9 2017, 4:31 PM
jgorbe created D29797: Update test/CodeGen/X86/sse-align-10.ll to use FileCheck instead of grep.
Feb 9 2017, 4:24 PM