[PATCH] D54195: Fix linker option for -fprofile-arcs -ftest-coverage

jessicah created this revision. Herald added a subscriber: cfe-commits. Linux toolchain accidentally added "-u__llvm_runtime_variable" when "-fprofile-arcs -ftest-coverage", this is not added when "--coverage" option is used. Using "-u__llvm_runtime_variable" generates an empty default.profraw

[PATCH] D54169: [clang-tidy] Zircon ->

aaron.ballman added inline comments. Comment at: clang-tools-extra/clang-tidy/zircon/FblLimitsCheck.cpp:47 +SrcMgr::CharacteristicKind FileType) { + if (FileName == "fbl/limits.h") { +unsigned End = std::strcspn(SM.getCharacterData(HashLoc), "\n") + 1;

[PATCH] D54169: [clang-tidy] Zircon ->

juliehockett added inline comments. Comment at: clang-tools-extra/clang-tidy/zircon/FblLimitsCheck.cpp:47 +SrcMgr::CharacteristicKind FileType) { + if (FileName == "fbl/limits.h") { +unsigned End = std::strcspn(SM.getCharacterData(HashLoc), "\n") + 1;

[PATCH] D54169: [clang-tidy] Zircon ->

aaron.ballman added inline comments. Comment at: clang-tools-extra/clang-tidy/zircon/FblLimitsCheck.cpp:47 +SrcMgr::CharacteristicKind FileType) { + if (FileName == "fbl/limits.h") { +unsigned End = std::strcspn(SM.getCharacterData(HashLoc), "\n") + 1;

[PATCH] D54169: [clang-tidy] Zircon ->

juliehockett added inline comments. Comment at: clang-tools-extra/clang-tidy/zircon/FblLimitsCheck.cpp:47 +SrcMgr::CharacteristicKind FileType) { + if (FileName == "fbl/limits.h") { +unsigned End = std::strcspn(SM.getCharacterData(HashLoc), "\n") + 1;

[PATCH] D54168: [clang-tidy] Zircon fbl::move -> std::move

juliehockett updated this revision to Diff 172892. juliehockett marked 2 inline comments as done. https://reviews.llvm.org/D54168 Files: clang-tools-extra/clang-tidy/zircon/CMakeLists.txt clang-tools-extra/clang-tidy/zircon/FblMoveCheck.cpp

[PATCH] D54169: [clang-tidy] Zircon ->

juliehockett updated this revision to Diff 172891. juliehockett marked 5 inline comments as done. https://reviews.llvm.org/D54169 Files: clang-tools-extra/clang-tidy/zircon/CMakeLists.txt clang-tools-extra/clang-tidy/zircon/FblLimitsCheck.cpp

[PATCH] D54168: [clang-tidy] Zircon fbl::move -> std::move

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/docs/ReleaseNotes.rst:188 + + Suggests converting uses of ``fbl::move`` to + ``std::move``, and suggests inserting the header. Please try to fill as much as possible to 80 characters.

[PATCH] D54168: [clang-tidy] Zircon fbl::move -> std::move

juliehockett updated this revision to Diff 172889. juliehockett marked 4 inline comments as done. https://reviews.llvm.org/D54168 Files: clang-tools-extra/clang-tidy/zircon/CMakeLists.txt clang-tools-extra/clang-tidy/zircon/FblMoveCheck.cpp

[PATCH] D47687: [Sema] Missing -Wlogical-op-parentheses warnings in macros (PR18971)

vsapsai added a comment. OK. Good to know you are still working on it. In https://reviews.llvm.org/D47687#1271880, @Higuoxing wrote: > In https://reviews.llvm.org/D47687#1266893, @vsapsai wrote: > > > Sorry about the delay. The change seems to be correct but `ninja > > check-clang` reveals the

[PATCH] D54187: Add debuginfo-tests that use cdb on Windows

zturner added a comment. I think the only way to realistically make this work for all platforms would be to separate the source file from the input/output. The source file would be the test case, and if you wanted to support a different debugger you would need to supply a different

[PATCH] D54188: [Sema] Mark target of __attribute__((alias("target"))) used for C

nickdesaulniers created this revision. nickdesaulniers added a reviewer: rsmith. Herald added subscribers: cfe-commits, erik.pilkington. Prevents -Wunneeded-internal-delcaration warnings when the target has no other references. This occurs frequently in device drivers in the Linux kernel. Sema

[PATCH] D54187: Add debuginfo-tests that use cdb on Windows

rnk added a comment. In https://reviews.llvm.org/D54187#1289607, @aprantl wrote: > It would be nice if instead of having a set of windows-only tests, we could > wrap cdb similar to llgdb.py wraps LLDB, so these tests run on all platforms. > If the Windows debugger is just too different for

[PATCH] D54187: Add debuginfo-tests that use cdb on Windows

aprantl added a comment. It would be nice if instead of having a set of windows-only tests, we could wrap cdb similar to llgdb.py wraps LLDB, so these tests run on all platforms. If the Windows debugger is just too different for this to make sense, let me know.

[PATCH] D53329: Generate DIFile with main program if source is not available

yonghong-song added a comment. @dblaikie This is not blocking me yet since the main BTF support is not in llvm yet. I will be in a conference next week. I will dig into this issue the week after, to have a smaller test case and try to root cause a little further. Will update here as soon as I

[PATCH] D54187: Add debuginfo-tests that use cdb on Windows

rnk created this revision. rnk added reviewers: aprantl, zturner. Herald added subscribers: JDevlieghere, mgorny. This is an initial prototype of how we can run debugger integration tests on Windows. cdb and windbg share a command language and debugger engine. Visual Studio has its own, but we

[PATCH] D54168: [clang-tidy] Zircon fbl::move -> std::move

juliehockett added a comment. In https://reviews.llvm.org/D54168#1289128, @JonasToth wrote: > I think this check is ok in the current form, but does it really need to be > in upstream clang-tidy? You said its for migration only, so it is not > valuable for you for a long time either? So yes,

r346276 - [Darwin] Export new weak external symbols when compiling with coverage

Author: vedantk Date: Tue Nov 6 14:25:00 2018 New Revision: 346276 URL: http://llvm.org/viewvc/llvm-project?rev=346276=rev Log: [Darwin] Export new weak external symbols when compiling with coverage Some weak external symbols were added to the profile runtime in D49953, and on Darwin, these

[PATCH] D53692: [analyzer] Evaluate all non-checker config options before analysis

Szelethus updated this revision to Diff 172856. Szelethus set the repository for this revision to rC Clang. Szelethus added a comment. - Moved initializer functions to `CompilerInvocation.cpp`, like every other Options-like class. - The fields for options are no longer `Optional` - Fixed the

[PATCH] D53692: [analyzer] Evaluate all non-checker config options before analysis

Szelethus marked 2 inline comments as done. Szelethus added inline comments. Comment at: test/Analysis/analyzer-config.c:4-12 void bar() {} void foo() { // Call bar 33 times so max-times-inline-large is met and // min-blocks-for-inline-large is checked for (int i = 0;

[PATCH] D54183: [HIP] Change default optimization level to -O3

tra added a comment. I'm not convinced that nvcc's behavior is a good guide for clang's defaults. Considering that clang is not compatible with nvcc when it comes to command line options, whoever is using clang to compile CUDA already has to adjust command line options. Explicitly adding `-O3`

[PATCH] D54183: [HIP] Change default optimization level to -O3

yaxunl created this revision. yaxunl added a reviewer: tra. The default optimization level of nvcc is -O3. There are HIP applications which expect the default optimization level to be -O3. Most HIP applications use -O3, therefore making it default. https://reviews.llvm.org/D54183 Files:

[PATCH] D54141: [clang-tidy] add deduplication support for run-clang-tidy.py

steveire added a comment. In https://reviews.llvm.org/D54141#1289326, @JonasToth wrote: > > That said, would you agree to have the parser-based deduplication as an > developer-only optin solution for now? :) If you're suggesting proceeding with this regex based solution, I don't think

[PATCH] D53153: [OpenCL] Mark kernel functions with default visibility

scott.linder added a comment. I don't believe that is currently the case (the unrestricted linking of OCL code to OCL code via a dynamic linker), but we do have the notion of a static link step, followed by dynamic linking at runtime. The static link step is currently via IR, but we plan to

[PATCH] D54169: [clang-tidy] Zircon ->

aaron.ballman added inline comments. Comment at: clang-tools-extra/clang-tidy/zircon/FblLimitsCheck.cpp:47 +SrcMgr::CharacteristicKind FileType) { + if (FileName == "fbl/limits.h") { +unsigned End = std::strcspn(SM.getCharacterData(HashLoc), "\n") + 1;

[PATCH] D54168: [clang-tidy] Zircon fbl::move -> std::move

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/clang-tidy/zircon/FblMoveCheck.cpp:61 +// Add in the header. +if (auto IncludeFixit = +Inserter->CreateIncludeInsertion(SM.getFileID(Start), "utility", Please don't use auto

[PATCH] D54169: [clang-tidy] Zircon ->

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/clang-tidy/zircon/FblLimitsCheck.cpp:111 + // Add in the header, since we know this file uses it. + if (auto IncludeFixit = Inserter->CreateIncludeInsertion( + SM.getFileID(V->getLocation()),

r346266 - Don't use std::next() on an input iterator; NFC.

Author: aaronballman Date: Tue Nov 6 13:12:44 2018 New Revision: 346266 URL: http://llvm.org/viewvc/llvm-project?rev=346266=rev Log: Don't use std::next() on an input iterator; NFC. Instead, advance the old-fashioned way, as std::next() cannot be used on an input iterator until C++17.

[PATCH] D54141: [clang-tidy] add deduplication support for run-clang-tidy.py

JonasToth added a comment. > At least the clang-tidy quiet mode is trivial to implement. Maybe instead of > `--quiet` we could have `--stdout=` where `output_format` can > be one of `none`, `diag`, `yaml` and in the future possibly `json` (requested > here:

[PATCH] D53329: Generate DIFile with main program if source is not available

dblaikie added a comment. In https://reviews.llvm.org/D53329#1270035, @yonghong-song wrote: > Sure. Let me provide a little bit more context and what I want to achieve: > > . I have a tool, called bcc (https://github.com/iovisor/bcc) which uses > clang CompilerInvocation interface and >

[PATCH] D54141: [clang-tidy] add deduplication support for run-clang-tidy.py

steveire added a comment. In https://reviews.llvm.org/D54141#1288930, @JonasToth wrote: > > Do you understand the proposal now? > > Yes better, I was under the impression that `clang-apply-replaments` is run > on the end and the YAMLs are kept until then. Now its clear. > I assume

[PATCH] D54141: [clang-tidy] add deduplication support for run-clang-tidy.py

JonasToth added a comment. In https://reviews.llvm.org/D54141#1288930, @JonasToth wrote: > > Do you understand the proposal now? > > Yes better, I was under the impression that `clang-apply-replaments` is run > on the end and the YAMLs are kept until then. Now its clear. > I assume

[PATCH] D54171: [MS] Zero out ECX in __cpuid in intrin.h

This revision was automatically updated to reflect the committed changes. Closed by commit rL346265: [MS] Zero out ECX in __cpuid in intrin.h (authored by rnk, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D54171?vs=172811=172831#toc

r346265 - [MS] Zero out ECX in __cpuid in intrin.h

Author: rnk Date: Tue Nov 6 12:45:26 2018 New Revision: 346265 URL: http://llvm.org/viewvc/llvm-project?rev=346265=rev Log: [MS] Zero out ECX in __cpuid in intrin.h Summary: Some CPUID leafs depend on the value of ECX as well as EAX, but we left it uninitialized. Originally reported as

[PATCH] D54171: [MS] Zero out ECX in __cpuid in intrin.h

craig.topper accepted this revision. craig.topper added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D54171 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D49736: [Basic] Emit warning flag suggestion only in case there's existing flag *similar* to the unknown one

jkorous requested review of this revision. jkorous added a comment. I tried to move the getNearestOption() to it's only client - EmitUnknownDiagWarning() but it turned out to be a significant change because of clang/Basic/DiagnosticGroups.inc use in DiagnosticIDs.cpp. I suggest we leave that

[PATCH] D54169: [clang-tidy] Zircon ->

mcgrathr accepted this revision. mcgrathr added a comment. This revision is now accepted and ready to land. lgtm https://reviews.llvm.org/D54169 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D54172: [AST] Pack ArraySubscriptExpr and CallExpr

riccibruno created this revision. riccibruno added a reviewer: rsmith. riccibruno added a project: clang. Herald added a subscriber: cfe-commits. Use the newly available space in the bit-fields of `Stmt` to store some data from `ArraySubscriptExpr` and `CallExpr`. This saves a pointer for each of

[PATCH] D54171: [MS] Zero out ECX in __cpuid in intrin.h

rnk created this revision. rnk added reviewers: craig.topper, hans. Some CPUID leafs depend on the value of ECX as well as EAX, but we left it uninitialized. Originally reported as https://crbug.com/901547 https://reviews.llvm.org/D54171 Files: clang/lib/Headers/intrin.h

[PATCH] D54168: [clang-tidy] Zircon fbl::move -> std::move

JonasToth added a comment. I think this check is ok in the current form, but does it really need to be in upstream clang-tidy? You said its for migration only, so it is not valuable for you for a long time either? Comment at:

[PATCH] D54169: [clang-tidy] Zircon ->

juliehockett created this revision. juliehockett added reviewers: aaron.ballman, alexfh, hokein. juliehockett added a project: clang-tools-extra. Herald added subscribers: xazax.hun, mgorny. Adds a check to convert to std . This check is part of a set of migration checks as we prepare to move

[PATCH] D54168: [clang-tidy] Zircon fbl::move -> std::move

juliehockett created this revision. juliehockett added reviewers: aaron.ballman, alexfh, hokein. juliehockett added a project: clang-tools-extra. Herald added subscribers: xazax.hun, mgorny. Adds a check to convert fbl::move to std::move. This check is part of a set of migration checks as we

[PATCH] D54141: [clang-tidy] add deduplication support for run-clang-tidy.py

JonasToth added a comment. In https://reviews.llvm.org/D54141#1288993, @Eugene.Zelenko wrote: > Reducing log file size is good idea, but I think will be also good idea to > count duplicates. This will allow to concentrate clean-up efforts on place > where most of warnings originate. Places

[PATCH] D54166: [AST] Store the string data in StringLiteral in a trailing array of chars

riccibruno created this revision. riccibruno added a reviewer: rsmith. riccibruno added a project: clang. Herald added a subscriber: cfe-commits. Use the newly available space in the bit-fields of `Stmt` and store the string data in a trailing array of `char`s after the trailing array of

[PATCH] D54162: OpenCL: Don't warn on v printf modifier

Anastasia accepted this revision. Anastasia added a comment. This revision is now accepted and ready to land. LGTM! Thanks! Seems like a reasonable start. https://reviews.llvm.org/D54162 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D54162: OpenCL: Don't warn on v printf modifier

arsenm created this revision. arsenm added a reviewer: Anastasia. Herald added subscribers: yaxunl, wdng. This avoids spurious warnings, but could use a lot of work. For example the number of vector elements is not verified, and the passed value type is not checked.

[PATCH] D54141: [clang-tidy] add deduplication support for run-clang-tidy.py

Eugene.Zelenko added a comment. Reducing log file size is good idea, but I think will be also good idea to count duplicates. This will allow to concentrate clean-up efforts on place where most of warnings originate. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D54141

[PATCH] D54161: [AST] Pack DeclRefExpr, UnaryOperator, MemberExpr and BinaryOperator

riccibruno created this revision. riccibruno added a reviewer: rsmith. riccibruno added a project: clang. Herald added a subscriber: cfe-commits. Move some data to the newly available space in the bit-fields of `Stmt`. This cuts the size of `DeclRefExpr`, `UnaryOperator`, `MemberExpr` and

[PATCH] D53738: [Fixed Point Arithmetic] Fixed Point Addition

rjmccall added a comment. In https://reviews.llvm.org/D53738#1288372, @ebevhan wrote: > In https://reviews.llvm.org/D53738#1288333, @rjmccall wrote: > > > In https://reviews.llvm.org/D53738#1287123, @ebevhan wrote: > > > > > In https://reviews.llvm.org/D53738#1284213, @rjmccall wrote: > > > > >

Re: r342827 - Fix modules build with shared library.

Sorry for the unresponsiveness, I've been fire fighting for other stuffs. I'll look into this and try to get it fixed this week. On Tue, Nov 6, 2018 at 9:52 AM David Blaikie wrote: > Shuai - have you had a chance to look at this? > > On Mon, Oct 22, 2018 at 4:43 PM Richard Smith > wrote: > >>

[PATCH] D52421: [Sema] Diagnose parameter names that shadow inherited field names

lebedev.ri added a comment. In https://reviews.llvm.org/D52421#1288917, @aaron.ballman wrote: > In https://reviews.llvm.org/D52421#1288910, @lebedev.ri wrote: > > > ... > > > ... Ok, thank you for reassuring me that it is working as intended. https://reviews.llvm.org/D52421

[PATCH] D54141: [clang-tidy] add deduplication support for run-clang-tidy.py

JonasToth added a comment. > Do you understand the proposal now? Yes better, I was under the impression that `clang-apply-replaments` is run on the end and the YAMLs are kept until then. Now its clear. I assume `--issue-diags` produce the same result as the normal diagnostic engine. That could

Re: r342827 - Fix modules build with shared library.

Shuai - have you had a chance to look at this? On Mon, Oct 22, 2018 at 4:43 PM Richard Smith wrote: > On Mon, 22 Oct 2018 at 14:57, David Blaikie via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > >> Richard - any further thoughts here (re: layering/dependencies, etc)? >> Would love to

[PATCH] D54141: [clang-tidy] add deduplication support for run-clang-tidy.py

steveire added a comment. In https://reviews.llvm.org/D54141#1288851, @JonasToth wrote: > > So, running `clang-apply-replacements --issue-diags the_new_file.yaml` > > would issue the warnings/fixit hints by processing the yaml and issuing the > > diagnostics the way clang-tidy would have done

[PATCH] D52421: [Sema] Diagnose parameter names that shadow inherited field names

aaron.ballman added a comment. In https://reviews.llvm.org/D52421#1288910, @lebedev.ri wrote: > And now i'm having concerns. > > struct Base { > void* f; > }; > > struct Inherit : Base { > static void func(void* f) { // <- does 'f' *actually* shadow the 'f' in > the 'Base'?

[PATCH] D53488: [clang-tidy] Catching narrowing from double to float.

JonasToth added a comment. I think the check is really close. If the other reviewers want to take a look at it again, now is a good moment :) Comment at: clang-tidy/cppcoreguidelines/NarrowingConversionsCheck.cpp:178 + return; +// Conversions to unsigned integer are

[PATCH] D52421: [Sema] Diagnose parameter names that shadow inherited field names

lebedev.ri added a comment. And now i'm having concerns. struct Base { void* f; }; struct Inherit : Base { static void func(void* f) { // <- does 'f' *actually* shadow the 'f' in the 'Base'? You can't access that non-static member variable from static function. }

[PATCH] D54157: [clangd] [NFC] Fix clang-tidy warnings.

JonasToth accepted this revision. JonasToth added a comment. This revision is now accepted and ready to land. Fixing such warnings is ok without prior review. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D54157 ___ cfe-commits

[PATCH] D54156: [CodeComplete] Do not complete self-initializations

sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. :-) Repository: rC Clang https://reviews.llvm.org/D54156 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D54077: [clangd] Implemented DraftFileSystem

sammccall added a comment. In https://reviews.llvm.org/D54077#1288455, @LutsenkoDanil wrote: > I already made a patch which introduces such behavior (not uploaded here > yet), and looks like it works well with draft fs: diagnostics updates for > depended files in 'real-time' on typing for

[PATCH] D54157: [clangd] [NFC] Fix clang-tidy warnings.

kadircet created this revision. kadircet added reviewers: ioeric, sammccall, ilya-biryukov, hokein. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D54157 Files: clangd/index/FileIndex.cpp

[PATCH] D53488: [clang-tidy] Catching narrowing from double to float.

gchatelet updated this revision to Diff 172778. gchatelet marked 7 inline comments as done. gchatelet added a comment. - Addressing comments and enhancing diagnostics Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D53488 Files:

[PATCH] D54141: [clang-tidy] add deduplication support for run-clang-tidy.py

JonasToth added a comment. > Maybe my suggestion was not clear. The yaml file generated by clang-tidy > contains not only replacements, but all diagnostics, even without a fixit. > > So, running `clang-apply-replacements --issue-diags the_new_file.yaml` would > issue the warnings/fixit hints

[PATCH] D54141: [clang-tidy] add deduplication support for run-clang-tidy.py

steveire added a comment. In https://reviews.llvm.org/D54141#1288818, @JonasToth wrote: > Thank you for the comment! > > In https://reviews.llvm.org/D54141#1288809, @steveire wrote: > > > This feature seems like a good idea. I started writing it too some months > > ago, but then I changed

[PATCH] D54152: [OpenCL] Fix diagnostic message about overload candidates

Anastasia accepted this revision. Anastasia added a comment. This revision is now accepted and ready to land. LGTM! Thanks! Repository: rC Clang https://reviews.llvm.org/D54152 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D54141: [clang-tidy] add deduplication support for run-clang-tidy.py

JonasToth added a comment. Thank you for the comment! In https://reviews.llvm.org/D54141#1288809, @steveire wrote: > This feature seems like a good idea. I started writing it too some months > ago, but then I changed tactic and worked on distributing the refactor over > the network instead.

[PATCH] D54141: [clang-tidy] add deduplication support for run-clang-tidy.py

JonasToth added a comment. > - The output of clang-tidy diagnostic is YAML, and YAML is not an > space-efficient format (just for human readability). If you want to save > space further, we might consider using some compressed formats, e.g. > llvm::bitcode. Given the reduced YAML result

[PATCH] D54141: [clang-tidy] add deduplication support for run-clang-tidy.py

steveire added a comment. This feature seems like a good idea. I started writing it too some months ago, but then I changed tactic and worked on distributing the refactor over the network instead. As far as I know, your deduplication would not work with a distributed environment. However, it

[PATCH] D54156: [CodeComplete] Do not complete self-initializations

ilya-biryukov updated this revision to Diff 172773. ilya-biryukov added a comment. - Remove std::move, the target is const ref, so it does nothing (thanks, clang-tidy!) Repository: rC Clang https://reviews.llvm.org/D54156 Files: lib/Sema/SemaCodeComplete.cpp

[PATCH] D54156: [CodeComplete] Do not complete self-initializations

ilya-biryukov created this revision. ilya-biryukov added a reviewer: sammccall. Herald added a subscriber: arphaman. Removes references to initialized variable from the following completions: int x = ^; Handles only the trivial cases where the variable name is completed immediately at the

[PATCH] D52998: [benchmark] Disable exceptions in Microsoft STL

This revision was automatically updated to reflect the committed changes. Closed by commit rL346237: [benchmark] Disable exceptions in Microsoft STL (authored by eandrews, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

[PATCH] D54141: [clang-tidy] add deduplication support for run-clang-tidy.py

hokein added a comment. Thanks for the patch and nice improvements. Some initial thoughts: - The output of clang-tidy diagnostic is YAML, and YAML is not an space-efficient format (just for human readability). If you want to save space further, we might consider using some compressed formats,

[PATCH] D41005: Reuse preamble even if an unsaved file does not exist

nik added a comment. Coming back to this one, I see a failing test: PCHPreambleTest.ReparseWithOverriddenFileDoesNotInvalidatePreamble Note that PCHPreambleTest.ReparseWithOverriddenFileDoesNotInvalidatePreamble references the header paths in different ways ("//./header1.h" vs

[PATCH] D52034: [Clang] Add options -fprofile-filter-files and -fprofile-exclude-files to filter the files to instrument with gcov

calixte updated this revision to Diff 172761. calixte added a comment. Fix plural form of regex and just use ';' as separator. Repository: rC Clang https://reviews.llvm.org/D52034 Files: docs/UsersManual.rst include/clang/Driver/Options.td include/clang/Frontend/CodeGenOptions.h

[PATCH] D54152: [OpenCL] Fix diagnostic message about overload candidates

AlexeySachkov created this revision. AlexeySachkov added reviewers: asavonic, Anastasia. Herald added a subscriber: yaxunl. I wonder if there are some extension which need to be disabled to get overloadable candidate available. Repository: rC Clang https://reviews.llvm.org/D54152 Files:

[PATCH] D53701: [Analyzer] Record and process comparison of symbols instead of iterator positions in interator checkers

baloghadamsoftware added a comment. In https://reviews.llvm.org/D53701#1288258, @NoQ wrote: > Mmm, is it possible to detect adapters and inline them as an exception from > the rule? You can foresee a pretty complicated system of rules and exceptions > if we go down this path, but i believe

[PATCH] D52034: [Clang] Add options -Xclang -coverage-filter and -Xclang -coverage-exclude to filter the files to instrument with gcov

calixte updated this revision to Diff 172752. calixte added a comment. Change options names to -fprofile-exclude-files & -fprofile-filter-files and add some doc in UserManual. Repository: rC Clang https://reviews.llvm.org/D52034 Files: docs/UsersManual.rst

[PATCH] D54148: [NFC][Clang][Aarch64] Add missing test file

dnsampaio created this revision. dnsampaio added a reviewer: olista01. Herald added subscribers: cfe-commits, kristof.beyls, javed.absar. The commit https://reviews.llvm.org/rL345273 by @LukeCheeseman has a missing test file, see https://reviews.llvm.org/D51429. This patch adds the missing test

[PATCH] D54149: [Analyzer] [WIP] Standard C++ library functions checker for the std::find() family

baloghadamsoftware created this revision. baloghadamsoftware added a reviewer: NoQ. baloghadamsoftware added a project: clang. Herald added subscribers: donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, rnkovacs, szepet, whisperity. Herald added a reviewer: george.karpenkov. Repository: rC

[PATCH] D54109: [clang-query] continue querying even if files are skipped

Lekensteyn updated this revision to Diff 172747. Lekensteyn retitled this revision from "[clang-query] continue even if files are skipped" to "[clang-query] continue querying even if files are skipped". Lekensteyn added a comment. Changes: - Return 1 (instead of 0) if none of the files could be

Re: r345591 - [CodeGen] Disable the machine verifier on a ThinLTO test

Thanks for the suggestion, I think it’s reasonable since it’s all due to: * ICALL_BRANCH_FUNNEL: Bad machine code: Explicit definition marked as use https://bugs.llvm.org/show_bug.cgi?id=39436 . I’ll look into it. > On 5 Nov 2018, at 19:13, David

[PATCH] D53457: clang-cl: Add "/clang:" pass-through arg support.

hans accepted this revision. hans added a comment. This revision is now accepted and ready to land. Okay, looks good to me (with a small nit). Comment at: docs/UsersManual.rst:2852 /Brepro Emit an object file which can be reproduced over time +

[PATCH] D51729: [Tooling] JSONCompilationDatabasePlugin infers compile commands for missing files

Lekensteyn added a comment. In https://reviews.llvm.org/D51729#1288360, @sammccall wrote: > > I'm not entirely sure what to do here. The old behavior works great in > > cases where a complete database is available (produced by CMake). The new > > behavior might work better for clangd (?), but

Re: r346167 - [Driver] Reland again again: Default Android toolchains to libc++.

Just fyi: this broke Chrome's build of compiler-rt using tot Clang against the sysroot generated by NDK r16's (I think) make_standalong_toolchain.py, see https://crbug.com/902270 It looks like we'll just hack around it though. On Mon, Nov 5, 2018 at 9:57 PM, Dan Albert via cfe-commits wrote: >

[PATCH] D52984: [analyzer] Checker reviewer's checklist

Szelethus added a comment. Also, context is missing :) https://reviews.llvm.org/D52984 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D52984: [analyzer] Checker reviewer's checklist

Szelethus added a comment. We probably should also add an entry about some code conventions we use here, for example, the use of `auto` was debated recently when used with `SVal::getAs`, maybe something like this: - As an LLVM subproject, the code in the Static Analyzer should too follow the

[PATCH] D53334: [Frontend/Modules] Show diagnostics on prebuilt module configuration mismatch too

whisperity planned changes to this revision. whisperity added a comment. In https://reviews.llvm.org/D53334#1288057, @dblaikie wrote: > In https://reviews.llvm.org/D53334#1273877, @whisperity wrote: > > > @dblaikie I have created a test, but unfortunately `%clang_cpp` in LIT > > invokes `clang

[PATCH] D54105: [clangd] Deduplicate query scopes.

This revision was automatically updated to reflect the committed changes. Closed by commit rL346224: [clangd] Deduplicate query scopes. (authored by ioeric, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D54105 Files:

[PATCH] D54077: [clangd] Implemented DraftFileSystem

ilya-biryukov added a comment. In https://reviews.llvm.org/D54077#1287289, @klimek wrote: > don't most IDEs show whether a file is saved or just modified? They do, but whenever you run the build from them, they will save all modified files before actually running it. In

[clang-tools-extra] r346224 - [clangd] Deduplicate query scopes.

Author: ioeric Date: Tue Nov 6 03:17:40 2018 New Revision: 346224 URL: http://llvm.org/viewvc/llvm-project?rev=346224=rev Log: [clangd] Deduplicate query scopes. Summary: For example, when anonymous namespace is present, duplicated namespaces might be generated for the enclosing namespace.

[PATCH] D52795: [analyzer][PlistMacroExpansion] Part 3.: Macro arguments are expanded

whisperity added inline comments. Comment at: lib/StaticAnalyzer/Core/PlistDiagnostics.cpp:1018 +auto It = CurrExpArgTokens.begin(); +while (It != CurrExpArgTokens.end()) { + if (It->isNot(tok::identifier)) { xazax.hun wrote: > Maybe a for loop mor

[PATCH] D54106: [clangd] Limit the index-returned results in dexp.

ioeric added inline comments. Comment at: clangd/index/dex/dexp/Dexp.cpp:166 + cl::init(10), + cl::desc("Max results to display. This flag is only meaningful when -name" + " is set."), Maybe `The max number of symbols with the same name

[PATCH] D54111: [clang-format] Do not treat the asm clobber [ as ObjCExpr

krasimir updated this revision to Diff 172738. krasimir added a comment. - Address review comments Repository: rC Clang https://reviews.llvm.org/D54111 Files: lib/Format/TokenAnnotator.cpp unittests/Format/FormatTest.cpp Index: unittests/Format/FormatTest.cpp

[PATCH] D54105: [clangd] Deduplicate query scopes.

ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land. LGTM Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D54105 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D53933: [clangd] Get rid of QueryScopes.empty() == AnyScope special case.

This revision was automatically updated to reflect the committed changes. Closed by commit rCTE346223: [clangd] Get rid of QueryScopes.empty() == AnyScope special case. (authored by ioeric, committed by ). Changed prior to commit: https://reviews.llvm.org/D53933?vs=172735=172737#toc

[PATCH] D54106: [clangd] Limit the index-returned results in dexp.

hokein updated this revision to Diff 172736. hokein added a comment. Update based on the offline discussion. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D54106 Files: clangd/index/dex/dexp/Dexp.cpp Index: clangd/index/dex/dexp/Dexp.cpp

[clang-tools-extra] r346223 - [clangd] Get rid of QueryScopes.empty() == AnyScope special case.

Author: ioeric Date: Tue Nov 6 03:08:17 2018 New Revision: 346223 URL: http://llvm.org/viewvc/llvm-project?rev=346223=rev Log: [clangd] Get rid of QueryScopes.empty() == AnyScope special case. Reviewers: sammccall Reviewed By: sammccall Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman,

[PATCH] D53933: [clangd] Get rid of QueryScopes.empty() == AnyScope special case.

ioeric updated this revision to Diff 172735. ioeric added a comment. - rebase Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D53933 Files: clangd/FindSymbols.cpp clangd/index/Index.h clangd/index/MemIndex.cpp clangd/index/dex/Dex.cpp unittests/clangd/DexTests.cpp

[clang-tools-extra] r346221 - [clangd] auto-index stores symbols per-file instead of per-TU.

Author: ioeric Date: Tue Nov 6 02:55:21 2018 New Revision: 346221 URL: http://llvm.org/viewvc/llvm-project?rev=346221=rev Log: [clangd] auto-index stores symbols per-file instead of per-TU. Summary: This allows us to deduplicate header symbols across TUs. File digests are collects when

[PATCH] D53433: [clangd] auto-index stores symbols per-file instead of per-TU.

This revision was automatically updated to reflect the committed changes. Closed by commit rL346221: [clangd] auto-index stores symbols per-file instead of per-TU. (authored by ioeric, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D53433

  1   2   >