bruno (Bruno Cardoso Lopes)
User

Projects

User does not belong to any projects.

User Details

User Since
Aug 15 2014, 7:40 AM (222 w, 5 d)

Recent Activity

Thu, Nov 15

bruno accepted D54245: [VFS] Implement `RedirectingFileSystem::getRealPath`..

LGTM

Thu, Nov 15, 1:26 PM

Tue, Nov 13

bruno accepted D54503: [HeaderSearch] loadSubdirectoryModuleMaps should respect -working-directory.

LGTM

Tue, Nov 13, 5:00 PM

Thu, Nov 8

bruno added a comment to D54245: [VFS] Implement `RedirectingFileSystem::getRealPath`..

Hi Volodymyr. Does this behavior changed after your VFS fallback change?

Thu, Nov 8, 12:01 PM

Wed, Oct 24

bruno added a reviewer for D53465: [VFS] Add support for "no_push" to VFS recursive iterators.: vsapsai.
Wed, Oct 24, 6:42 PM

Tue, Oct 23

bruno accepted D50539: [VFS] Add property 'fallthrough' that controls fallback to real file system..

LGTM with some minor changes.

Tue, Oct 23, 11:28 AM
bruno accepted D53228: [VFS] Remove 'ignore-non-existent-contents' attribute for YAML-based VFS..

LGTM.

Tue, Oct 23, 11:20 AM
bruno added a comment to D53522: [Frontend] Include module map header declaration in dependency file output.

Hi Erik, thanks for improving this. Comments below.

Tue, Oct 23, 11:16 AM

Sep 20 2018

bruno added a comment to D50539: [VFS] Add property 'fallthrough' that controls fallback to real file system..

Hi Volodymyr,

Sep 20 2018, 1:29 PM
bruno accepted D52253: Fix an assert in the implementation of -Wquoted-include-in-framework-header.

LGTM with one more small change.

Sep 20 2018, 11:45 AM

Sep 18 2018

bruno added a comment to D52253: Fix an assert in the implementation of -Wquoted-include-in-framework-header.

Hi Erik,

Sep 18 2018, 6:35 PM
bruno committed rL342499: [Modules] Add platform and environment features to requires clause.
[Modules] Add platform and environment features to requires clause
Sep 18 2018, 10:15 AM
bruno committed rC342499: [Modules] Add platform and environment features to requires clause.
[Modules] Add platform and environment features to requires clause
Sep 18 2018, 10:15 AM
bruno closed D51910: [Modules] Add platform feature to requires clause.
Sep 18 2018, 10:15 AM
bruno closed D51910: [Modules] Add platform feature to requires clause.
Sep 18 2018, 10:15 AM

Sep 17 2018

bruno updated the diff for D51910: [Modules] Add platform feature to requires clause.

Update patch after review.

Sep 17 2018, 6:22 PM
bruno added inline comments to D51910: [Modules] Add platform feature to requires clause.
Sep 17 2018, 2:43 PM

Sep 14 2018

bruno updated the diff for D51910: [Modules] Add platform feature to requires clause.

Addressed Adrian's review. Added support to consider the environment and well the combination platform-environment. @aprantl also handled multiple variants of simulator combinations.

Sep 14 2018, 4:33 PM

Sep 10 2018

bruno committed rC341902: [Modules] Add imported modules to the output of -module-file-info.
[Modules] Add imported modules to the output of -module-file-info
Sep 10 2018, 10:18 PM
bruno committed rL341902: [Modules] Add imported modules to the output of -module-file-info.
[Modules] Add imported modules to the output of -module-file-info
Sep 10 2018, 10:18 PM
bruno created D51910: [Modules] Add platform feature to requires clause.
Sep 10 2018, 9:49 PM

Aug 17 2018

bruno committed rL340119: Revert "Add a basic integration test for C++ smart pointers".
Revert "Add a basic integration test for C++ smart pointers"
Aug 17 2018, 9:17 PM
bruno committed rL340117: Revert "[analyzer] [NFC] Split up RetainSummaryManager from RetainCountChecker".
Revert "[analyzer] [NFC] Split up RetainSummaryManager from RetainCountChecker"
Aug 17 2018, 8:23 PM
bruno committed rC340117: Revert "[analyzer] [NFC] Split up RetainSummaryManager from RetainCountChecker".
Revert "[analyzer] [NFC] Split up RetainSummaryManager from RetainCountChecker"
Aug 17 2018, 8:22 PM
bruno accepted D50870: Close FileEntries of cached files in ModuleManager::addModule()..

Thanks for working on this Adrian, LGTM.

Aug 17 2018, 6:16 PM

Aug 14 2018

bruno committed rL339700: Revert "[DebugInfo] Generate DWARF debug information for labels. (Fix leak….
Revert "[DebugInfo] Generate DWARF debug information for labels. (Fix leak…
Aug 14 2018, 10:55 AM

Aug 7 2018

bruno accepted D50118: [VFS] Unify iteration code for VFSFromYamlDirIterImpl, NFC intended..

Thanks for the clean up! LGTM

Aug 7 2018, 11:18 AM
bruno accepted D49518: [VFS] Emit an error when a file isn't located in any directory..

LGTM

Aug 7 2018, 11:11 AM

Jul 20 2018

bruno committed rL337555: [www] Add CodeCompass and CodeChecker to Clang Related Projects page.
[www] Add CodeCompass and CodeChecker to Clang Related Projects page
Jul 20 2018, 7:51 AM
bruno committed rC337555: [www] Add CodeCompass and CodeChecker to Clang Related Projects page.
[www] Add CodeCompass and CodeChecker to Clang Related Projects page
Jul 20 2018, 7:51 AM

Jul 19 2018

bruno added a comment to D49518: [VFS] Emit an error when a file isn't located in any directory..

Hi Volodymyr, thanks for improving this.

Jul 19 2018, 6:50 AM
bruno accepted D48786: [Preprocessor] Stop entering included files after hitting a fatal error..

Thanks for working on this.

Jul 19 2018, 5:49 AM
bruno committed rC337447: [PCH+Modules] Load -fmodule-map-file content before including PCHs.
[PCH+Modules] Load -fmodule-map-file content before including PCHs
Jul 19 2018, 5:38 AM
bruno committed rL337447: [PCH+Modules] Load -fmodule-map-file content before including PCHs.
[PCH+Modules] Load -fmodule-map-file content before including PCHs
Jul 19 2018, 5:37 AM
bruno closed D48685: [PCH+Modules] Load -fmodule-map-file content before including PCHs.
Jul 19 2018, 5:37 AM

Jul 18 2018

bruno committed rL337430: Reapply r336660: [Modules] Autoload subdirectory modulemaps with specific….
Reapply r336660: [Modules] Autoload subdirectory modulemaps with specific…
Jul 18 2018, 4:26 PM
bruno committed rC337430: Reapply r336660: [Modules] Autoload subdirectory modulemaps with specific….
Reapply r336660: [Modules] Autoload subdirectory modulemaps with specific…
Jul 18 2018, 4:26 PM

Jul 12 2018

bruno committed rL336920: Revert "[modules] Fix 37878; Autoload subdirectory modulemaps with specific….
Revert "[modules] Fix 37878; Autoload subdirectory modulemaps with specific…
Jul 12 2018, 10:44 AM
bruno committed rC336920: Revert "[modules] Fix 37878; Autoload subdirectory modulemaps with specific….
Revert "[modules] Fix 37878; Autoload subdirectory modulemaps with specific…
Jul 12 2018, 10:44 AM

Jul 6 2018

bruno added a comment to D48685: [PCH+Modules] Load -fmodule-map-file content before including PCHs.

Ping!

Jul 6 2018, 10:59 AM

Jun 29 2018

bruno committed rL336031: Add protocol redefinition to the current scope/context.
Add protocol redefinition to the current scope/context
Jun 29 2018, 5:54 PM
bruno committed rC336031: Add protocol redefinition to the current scope/context.
Add protocol redefinition to the current scope/context
Jun 29 2018, 5:54 PM
bruno closed D47297: [Modules][ObjC] Add protocol redefinition to the current scope/context.
Jun 29 2018, 5:54 PM
bruno closed D47297: [Modules][ObjC] Add protocol redefinition to the current scope/context.
Jun 29 2018, 5:54 PM
bruno added inline comments to D47297: [Modules][ObjC] Add protocol redefinition to the current scope/context.
Jun 29 2018, 11:37 AM
bruno added a comment to D47297: [Modules][ObjC] Add protocol redefinition to the current scope/context.

Ping!

Jun 29 2018, 10:11 AM

Jun 28 2018

bruno accepted D48736: [frontend] Don't include the C++ stdlib for -x assembler-with-cpp.

LGTM

Jun 28 2018, 3:57 PM
bruno added inline comments to D48736: [frontend] Don't include the C++ stdlib for -x assembler-with-cpp.
Jun 28 2018, 2:58 PM
bruno accepted D48367: [modules] Fix 37878; Autoload subdirectory modulemaps with specific LangOpts.

Thanks for working on this, LGTM

Jun 28 2018, 11:00 AM
bruno accepted D47118: [modules] Print input files when -module-file-info file switch is passed..

It looks like the diagnostic options and few others were moved out from the control block in r297655 by Manman Ren. The header files are still part of the control block. The best I can do is buffer that information and print it out last. Is that what we want?

Jun 28 2018, 9:47 AM

Jun 27 2018

bruno created D48685: [PCH+Modules] Load -fmodule-map-file content before including PCHs.
Jun 27 2018, 4:45 PM
bruno committed rL335780: [Modules][ObjC] Warn on the use of '@import' in framework headers.
[Modules][ObjC] Warn on the use of '@import' in framework headers
Jun 27 2018, 1:34 PM
bruno committed rC335780: [Modules][ObjC] Warn on the use of '@import' in framework headers.
[Modules][ObjC] Warn on the use of '@import' in framework headers
Jun 27 2018, 1:34 PM

Jun 25 2018

bruno closed D47301: Warning for framework include violation from Headers to PrivateHeaders.

Committed r335542

Jun 25 2018, 3:36 PM
bruno committed rL335543: Fix tests from r335542 to use %hmaptool.
Fix tests from r335542 to use %hmaptool
Jun 25 2018, 3:30 PM
bruno committed rC335543: Fix tests from r335542 to use %hmaptool.
Fix tests from r335542 to use %hmaptool
Jun 25 2018, 3:30 PM
bruno committed rC335542: Warning for framework include violation from Headers to PrivateHeaders.
Warning for framework include violation from Headers to PrivateHeaders
Jun 25 2018, 3:29 PM
bruno committed rL335542: Warning for framework include violation from Headers to PrivateHeaders.
Warning for framework include violation from Headers to PrivateHeaders
Jun 25 2018, 3:29 PM
bruno added inline comments to D48367: [modules] Fix 37878; Autoload subdirectory modulemaps with specific LangOpts.
Jun 25 2018, 2:45 PM

Jun 22 2018

bruno added a comment to D47301: Warning for framework include violation from Headers to PrivateHeaders.

Ping!

Jun 22 2018, 2:10 PM
bruno committed rC335375: Re-apply: Warning for framework headers using double quote includes.
Re-apply: Warning for framework headers using double quote includes
Jun 22 2018, 11:10 AM
bruno committed rL335375: Re-apply: Warning for framework headers using double quote includes.
Re-apply: Warning for framework headers using double quote includes
Jun 22 2018, 11:09 AM

Jun 21 2018

bruno committed rC335295: Re-apply: Add python tool to dump and construct header maps.
Re-apply: Add python tool to dump and construct header maps
Jun 21 2018, 2:50 PM
bruno committed rL335295: Re-apply: Add python tool to dump and construct header maps.
Re-apply: Add python tool to dump and construct header maps
Jun 21 2018, 2:50 PM
bruno added inline comments to D48367: [modules] Fix 37878; Autoload subdirectory modulemaps with specific LangOpts.
Jun 21 2018, 9:42 AM

Jun 20 2018

bruno committed rC335196: Revert "Add python tool to dump and construct header maps".
Revert "Add python tool to dump and construct header maps"
Jun 20 2018, 6:28 PM
bruno committed rL335196: Revert "Add python tool to dump and construct header maps".
Revert "Add python tool to dump and construct header maps"
Jun 20 2018, 6:28 PM
bruno committed rC335195: Revert "Warning for framework headers using double quote includes".
Revert "Warning for framework headers using double quote includes"
Jun 20 2018, 6:28 PM
bruno committed rL335195: Revert "Warning for framework headers using double quote includes".
Revert "Warning for framework headers using double quote includes"
Jun 20 2018, 6:28 PM
bruno committed rL335194: Revert "Fix hmaptool cmake file to work on Windows".
Revert "Fix hmaptool cmake file to work on Windows"
Jun 20 2018, 6:28 PM
bruno committed rC335194: Revert "Fix hmaptool cmake file to work on Windows".
Revert "Fix hmaptool cmake file to work on Windows"
Jun 20 2018, 6:28 PM
bruno updated the diff for D47301: Warning for framework include violation from Headers to PrivateHeaders.

Address Volodymyr's comments

Jun 20 2018, 4:22 PM
bruno committed rL335190: Fix hmaptool cmake file to work on Windows.
Fix hmaptool cmake file to work on Windows
Jun 20 2018, 4:13 PM
bruno committed rC335190: Fix hmaptool cmake file to work on Windows.
Fix hmaptool cmake file to work on Windows
Jun 20 2018, 4:13 PM
bruno updated subscribers of D46485: Add python tool to dump and construct header maps.

Hi Stella,

Jun 20 2018, 3:53 PM
bruno committed rC335184: Warning for framework headers using double quote includes.
Warning for framework headers using double quote includes
Jun 20 2018, 3:16 PM
bruno committed rL335184: Warning for framework headers using double quote includes.
Warning for framework headers using double quote includes
Jun 20 2018, 3:16 PM
bruno closed D47157: Warning for framework headers using double quote includes.
Jun 20 2018, 3:16 PM
bruno committed rL335181: Fix WasmEHFuncInfo.h to include what it uses.
Fix WasmEHFuncInfo.h to include what it uses
Jun 20 2018, 2:48 PM
bruno committed rL335177: Add python tool to dump and construct header maps.
Add python tool to dump and construct header maps
Jun 20 2018, 2:21 PM
bruno committed rC335177: Add python tool to dump and construct header maps.
Add python tool to dump and construct header maps
Jun 20 2018, 2:21 PM
bruno closed D46485: Add python tool to dump and construct header maps.
Jun 20 2018, 2:21 PM
bruno updated the diff for D46485: Add python tool to dump and construct header maps.

Update after Richard's review, fix python3 compatibility and check for zero sized string table

Jun 20 2018, 1:14 PM
bruno added a comment to D48367: [modules] Fix 37878; Autoload subdirectory modulemaps with specific LangOpts.

Hi Yuka, thanks for working on this.

Jun 20 2018, 9:52 AM

Jun 19 2018

bruno accepted D48297: [Darwin] Add a warning for missing include path for libstdc++.

LGTM

Jun 19 2018, 10:57 AM

Jun 18 2018

bruno added inline comments to D48297: [Darwin] Add a warning for missing include path for libstdc++.
Jun 18 2018, 2:26 PM

Jun 15 2018

bruno committed rC334859: [Modules] Improve .Private fix-its to handle 'explicit' and 'framework'.
[Modules] Improve .Private fix-its to handle 'explicit' and 'framework'
Jun 15 2018, 1:17 PM
bruno committed rL334859: [Modules] Improve .Private fix-its to handle 'explicit' and 'framework'.
[Modules] Improve .Private fix-its to handle 'explicit' and 'framework'
Jun 15 2018, 1:17 PM

Jun 14 2018

bruno committed rC334747: [CMAKE][c-index-test] Honor CMAKE_OSX_SYSROOT to compute include dir for libxml2.
[CMAKE][c-index-test] Honor CMAKE_OSX_SYSROOT to compute include dir for libxml2
Jun 14 2018, 11:24 AM
bruno committed rL334747: [CMAKE][c-index-test] Honor CMAKE_OSX_SYSROOT to compute include dir for libxml2.
[CMAKE][c-index-test] Honor CMAKE_OSX_SYSROOT to compute include dir for libxml2
Jun 14 2018, 11:24 AM
bruno committed rL334746: [CMAKE] Honor CMAKE_OSX_SYSROOT to compute include dir for libxml2.
[CMAKE] Honor CMAKE_OSX_SYSROOT to compute include dir for libxml2
Jun 14 2018, 11:24 AM

Jun 11 2018

bruno accepted D47929: Add modules support for lldb headers in include/.

LGTM

Jun 11 2018, 11:35 AM

Jun 8 2018

bruno added a comment to D47929: Add modules support for lldb headers in include/.

Very nice!

Jun 8 2018, 9:49 AM

May 31 2018

bruno committed rL333718: [Modules] Warning for module declarations lacking 'framework' qualifier.
[Modules] Warning for module declarations lacking 'framework' qualifier
May 31 2018, 6:30 PM
bruno committed rC333718: [Modules] Warning for module declarations lacking 'framework' qualifier.
[Modules] Warning for module declarations lacking 'framework' qualifier
May 31 2018, 6:30 PM
bruno added a reviewer for D47297: [Modules][ObjC] Add protocol redefinition to the current scope/context: arphaman.
May 31 2018, 5:19 PM
bruno added a comment to D46485: Add python tool to dump and construct header maps.

Without this patch, we would have to add binary header maps for tests in D47157 and D47301, which I would like to avoid.

May 31 2018, 5:18 PM
bruno added inline comments to D47301: Warning for framework include violation from Headers to PrivateHeaders.
May 31 2018, 2:58 PM
bruno updated the diff for D47301: Warning for framework include violation from Headers to PrivateHeaders.

Update patch after changes to D47157. Also address some of Volodymyr feedback.

May 31 2018, 2:52 PM
bruno added inline comments to D47157: Warning for framework headers using double quote includes.
May 31 2018, 2:41 PM
bruno updated the diff for D47157: Warning for framework headers using double quote includes.

Update after Duncan's review: remove header name from the warning message (since it's already in the fixit)

May 31 2018, 2:40 PM