[PATCH] D72755: [RISCV] Pass target-abi via module flag metadata

This revision was automatically updated to reflect the committed changes. Closed by commit rGe15fb06e2d0a: [RISCV] Pass target-abi via module flag metadata (authored by Zakk Chen ). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D

[clang] e15fb06 - [RISCV] Pass target-abi via module flag metadata

Author: Zakk Chen Date: 2020-01-20T23:30:54-08:00 New Revision: e15fb06e2d0a068de549464d72081811e7fac612 URL: https://github.com/llvm/llvm-project/commit/e15fb06e2d0a068de549464d72081811e7fac612 DIFF: https://github.com/llvm/llvm-project/commit/e15fb06e2d0a068de549464d72081811e7fac612.diff LOG

[PATCH] D69868: Allow "callbr" to return non-void values

void updated this revision to Diff 239231. void added a comment. Herald added a subscriber: MatzeB. Split the machine basic block after an INLINEASM_BR instruction that has outputs. The copies then end up in a separate block and the back end doesn't have to deal with COPY instructions between two

[PATCH] D44609: [clang-format] New option BeforeLambdaBody to manage lambda line break inside function parameter call (in Allman style)

MyDeveloperDay added a comment. I think if you can fix the missing documentation it looks pretty good to go. I do like the use of the static functions and the extensive test cases.. thank you. Comment at: clang/lib/Format/ContinuationIndenter.cpp:1442 +// Search for any p

[PATCH] D44609: [Clang-Format] New option BeforeLambdaBody to manage lambda line break inside function parameter call (in Allman style)

MyDeveloperDay added a comment. Please can you regenerate the ClangFormatStyleOptions.rst using docs/tools/dump_style.py and add an entry into the clang release notes in the clang-format section Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D44609/

[PATCH] D72841: [RFC] Add support for pragma float_control, to control precision and exception behavior at the source level

sepavloff added inline comments. Comment at: clang/lib/Parse/ParsePragma.cpp:2537 +if (!Actions.CurContext->isTranslationUnit()) { +//FIXME this seems to be the wrong way to check file-scope +//since the token immediately following a function definition Proba

[PATCH] D72638: [clangd] Fix rename for explicit destructor calls

This revision was automatically updated to reflect the committed changes. Closed by commit rG38bdb94120b7: [clangd] Fix rename for explicit destructor calls (authored by kbobyrev). Changed prior to commit: https://reviews.llvm.org/D72638?vs=239182&id=239227#toc Repository: rG LLVM Github Mon

[clang-tools-extra] 38bdb94 - [clangd] Fix rename for explicit destructor calls

Author: Kirill Bobyrev Date: 2020-01-21T05:33:39+01:00 New Revision: 38bdb94120b76f8f79cd27d721892673e573895a URL: https://github.com/llvm/llvm-project/commit/38bdb94120b76f8f79cd27d721892673e573895a DIFF: https://github.com/llvm/llvm-project/commit/38bdb94120b76f8f79cd27d721892673e573895a.diff

[PATCH] D72829: Implement -fsemantic-interposition

MaskRay added inline comments. Comment at: llvm/lib/IR/Globals.cpp:101 +return true; + return isInterposableLinkage(getLinkage()); +} MaskRay wrote: > Checking `isInterposableLinkage(getLinkage())` first may be more efficient. `if (!isInterposableLinkage(get

[PATCH] D72829: Implement -fsemantic-interposition

MaskRay added a comment. The logic looks good to me. Linkages which were not interposable before and can be interposable now: available_externally, linkonce_odr, weak_odr, external, and appending. `isDefinitionExact` returned true for external before, and can return false if SemanticInterposit

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62040 tests passed, 1 failed and 783 were skipped. failed: Clang Tools.clang-tidy/checkers/readability-identifier-naming-member-decl-usage.cpp {icon question-circle color=gray} clang-tidy: unknown. {icon times

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

njames93 updated this revision to Diff 239218. njames93 added a comment. - rebase master figure out the llvm pre-merge lint test fail Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73052/new/ https://reviews.llvm.org/D73052 Files: clang-tools-ext

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62026 tests passed, 1 failed and 783 were skipped. failed: Clang Tools.clang-tidy/checkers/readability-identifier-naming-member-decl-usage.cpp {icon question-circle color=gray} clang-tidy: unknown. {icon times

[PATCH] D73072: [Driver][CodeGen] Support -fpatchable-function-entry=N,M where M>0

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62040 tests passed, 2 failed and 783 were skipped. failed: Clang.CodeGen/patchable-function-entry.c failed: Clang.Driver/fpatchable-function-entry.c {icon question-circle color=gray} clang-tidy: unknown. {ico

[PATCH] D71913: [LTO/WPD] Enable aggressive WPD under LTO option

tejohnson updated this revision to Diff 239214. tejohnson marked 2 inline comments as done. tejohnson added a comment. Address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71913/new/ https://reviews.llvm.org/D71913 Files: clang/lib/Cod

[PATCH] D71913: [LTO/WPD] Enable aggressive WPD under LTO option

tejohnson marked 4 inline comments as done. tejohnson added inline comments. Comment at: llvm/test/Transforms/WholeProgramDevirt/import-indir.ll:2 ; Test that we correctly import an indir resolution for type identifier "typeid1". -; RUN: opt -S -wholeprogramdevirt -wholeprogram

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62026 tests passed, 1 failed and 783 were skipped. failed: Clang Tools.clang-tidy/checkers/readability-identifier-naming-member-decl-usage.cpp {icon question-circle color=gray} clang-tidy: unknown. {icon times

[PATCH] D67678: PR17164: Change clang's default behavior from -flax-vector-conversions=all to -flax-vector-conversions=integer.

hctim added a comment. This change broke the sanitizer buildbots and was reverted here . Please see here for more information: /b/sanitizer-x86_64-li

[PATCH] D73072: [Driver][CodeGen] Support -fpatchable-function-entry=N,M where M>0

MaskRay created this revision. MaskRay added reviewers: aaron.ballman, craig.topper, nickdesaulniers, nsz, ostannard, peter.smith. Herald added a project: clang. Herald added a subscriber: cfe-commits. MaskRay added a parent revision: D73071: [X86] Support -fpatchable-function-entry=N,M where M>0

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

njames93 updated this revision to Diff 239208. njames93 added a comment. - Added some sanity tests that fix weird crashes - Added support for CXXMethodDecls that are referenced with depended scope member expr Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llv

[PATCH] D49793: [AArch64] - return address signing

hctim added a comment. Herald added a project: LLVM. This change broke the sanitizer buildbots. Please see here for more information: /b/sanitizer-x86_64-linux/build/llvm-pr

[clang] edd4398 - Revert "PR17164: Change clang's default behavior from -flax-vector-conversions=all to -flax-vector-conversions=integer."

Author: Mitch Phillips Date: 2020-01-20T16:34:09-08:00 New Revision: edd4398f4cd33a305afbca76ac4e6590e9337f4d URL: https://github.com/llvm/llvm-project/commit/edd4398f4cd33a305afbca76ac4e6590e9337f4d DIFF: https://github.com/llvm/llvm-project/commit/edd4398f4cd33a305afbca76ac4e6590e9337f4d.diff

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

njames93 added a comment. hmmm annoyingly this change causes a crash in a side project I'm creating, gotta go bug hunting now Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73052/new/ https://reviews.llvm.org/D73052 _

[PATCH] D72638: [clangd] Fix rename for explicit destructor calls

merge_guards_bot added a comment. {icon question-circle color=gray} Unit tests: unknown. {icon question-circle color=gray} clang-tidy: unknown. {icon check-circle color=green} clang-format: pass. Build artifacts : diff.json

[PATCH] D72736: [AIX] Add improved interface for retrieving load module paths

Jonathan.Crowther updated this revision to Diff 239198. Jonathan.Crowther added a comment. Addressed comments by moving `getLoadModuleFilnameForFunction` to `llvm/lib/Support/SystemUtils.cpp` CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72736/new/ https://reviews.llvm.org/D72736 File

[PATCH] D71566: New checks for fortified sprintf

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62028 tests passed, 0 failed and 783 were skipped. {icon question-circle color=gray} clang-tidy: unknown. {icon times-circle color=red} clang-format: fail. Please format your changes with clang-format by runnin

[PATCH] D69978: Separately track input and output denormal mode

arsenm updated this revision to Diff 239196. arsenm added a comment. Herald added a subscriber: kerbowa. Rebase CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69978/new/ https://reviews.llvm.org/D69978 Files: clang/include/clang/Basic/CodeGenOptions.h clang/include/clang/Driver/Tool

[PATCH] D71566: New checks for fortified sprintf

serge-sans-paille updated this revision to Diff 239197. serge-sans-paille marked an inline comment as done. serge-sans-paille added a comment. Finer grain lower bound computations, and simpler control-flow for the checking function. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTI

[PATCH] D72638: [clangd] Fix rename for explicit destructor calls

merge_guards_bot added a comment. {icon question-circle color=gray} Unit tests: unknown. {icon question-circle color=gray} clang-tidy: unknown. {icon check-circle color=green} clang-format: pass. Build artifacts : diff.json

[PATCH] D28955: [analyzer] Enable support for symbolic extension/truncation

mikhail.ramalho updated this revision to Diff 239186. mikhail.ramalho added a comment. Herald added subscribers: llvm-commits, hiraditya. Herald added a project: LLVM. Two changes: - Moved the BoolAssignmentChecker changes to separate revision (D73062 ). - Rebase

[PATCH] D72638: [clangd] Fix rename for explicit destructor calls

kbobyrev updated this revision to Diff 239182. kbobyrev marked 2 inline comments as done. kbobyrev added a comment. Modify the test to address the comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72638/new/ https://reviews.llvm.org/D72638 F

[PATCH] D73062: [analyzer] Simplify BoolAssignmentChecker

mikhail.ramalho created this revision. mikhail.ramalho added a reviewer: NoQ. Herald added subscribers: cfe-commits, Charusso. Herald added a project: clang. mikhail.ramalho retitled this revision from "Simplify BoolAssignmentChecker" to "[analyzer] Simplify BoolAssignmentChecker". Herald added su

[PATCH] D72638: [clangd] Fix rename for explicit destructor calls

kbobyrev added inline comments. Comment at: clang-tools-extra/clangd/unittests/FindTargetTests.cpp:905 + public: + ~$1^Foo() {} + kadircet wrote: > this actually looks troublesome, since we are not returning the functiondecl > anymo

[PATCH] D73060: [Clang] Fix expansion of response files in -Wp after integrated-cc1 change

wenlei accepted this revision. wenlei added a comment. This revision is now accepted and ready to land. > Is that the desired behavior? This was the previous behavior but I'm not sure > it's the right behavior. Good point. It might be safer to keep that previous behavior though. Thanks for the

[PATCH] D73020: [Sema] Perform call checking when building CXXNewExpr

lebedev.ri updated this revision to Diff 239180. lebedev.ri added a comment. Not only do we need to 'insert' size implicit argument, but we may need to insert alignment implicit argument. And while for size we can only insert `nullptr`, for alignment we can actually insert the correct value. Rep

[PATCH] D68720: Support -fstack-clash-protection for x86

merge_guards_bot added a comment. {icon times-circle color=red} Unit tests: fail. 62035 tests passed, 2 failed and 783 were skipped. failed: Clang.CodeGen/stack-clash-protection.c failed: Clang.Misc/warning-flags.c {icon question-circle color=gray} clang-tidy: unknown. {icon times-circle c

[PATCH] D71124: [RISCV] support clang driver to select cpu

lenary added a comment. In D71124#1792216 , @khchen wrote: > The problem is how `-mcpu` interact with explicitly specified `-march` (or > target features). > > 1. in GCC, -mcpu is only used to chose the pipeline model, I think you mean "in GCC, `-mtune`

[PATCH] D28955: [analyzer] Enable support for symbolic extension/truncation

mikhail.ramalho added a comment. I can give some help here, but my time is quite limited as well :/ Let me at least rebase against master. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D28955/new/ https://reviews.llvm.org/D28955 ___ cfe-com

[PATCH] D73019: [Sema] Don't disallow placing `__attribute__((alloc_align(param_idx)))` on `std::align_val_t`-typed parameters

lebedev.ri updated this revision to Diff 239179. lebedev.ri added a comment. NFC, more tests. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73019/new/ https://reviews.llvm.org/D73019 Files: clang/lib/Sema/SemaDeclAttr.cpp clang/test/SemaCXX/st

[PATCH] D73056: [clangd] Fix DocumentOutline for concepts

This revision was automatically updated to reflect the committed changes. Closed by commit rGfb3d9153c01b: [clangd] Fix DocumentOutline for concepts (authored by kadircet). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73056/new/ https://reviews.llv

[PATCH] D73060: [Clang] Fix expansion of response files in -Wp after integrated-cc1 change

aganea created this revision. aganea added reviewers: hans, thakis, wenlei. aganea added a project: clang. Herald added a subscriber: cfe-commits. aganea edited the summary of this revision. After rGb4a99a061f517e60985667e39519f60186cbb469

[PATCH] D72932: [ARM] Follow AACPS standard for volatile bit-fields access width

dnsampaio added a comment. In D72932#1829716 , @ostannard wrote: > Why are you doing this in CodeGen, rather than adjusting the existing layout > code in CGRecordLowering? Doing it this way will result in > AdjustAAPCSBitfieldLValue being called for ever

[clang-tools-extra] fb3d915 - [clangd] Fix DocumentOutline for concepts

Author: Kadir Cetinkaya Date: 2020-01-20T20:20:27+01:00 New Revision: fb3d9153c01b9a560680465190d6ecd804e4c486 URL: https://github.com/llvm/llvm-project/commit/fb3d9153c01b9a560680465190d6ecd804e4c486 DIFF: https://github.com/llvm/llvm-project/commit/fb3d9153c01b9a560680465190d6ecd804e4c486.dif

[PATCH] D73056: [clangd] Fix DocumentOutline for concepts

kbobyrev accepted this revision. kbobyrev added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73056/new/ https://reviews.llvm.org/D73056 ___

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62027 tests passed, 0 failed and 783 were skipped. {icon question-circle color=gray} clang-tidy: unknown. {icon times-circle color=red} clang-format: fail. Please format your changes with clang-format by runnin

LLVM buildmaster will be updated and restarted tonight

Hello everyone, LLVM buildmaster will be updated and restarted after 5PM Pacific time today. Thanks Galina ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D68720: Support -fstack-clash-protection for x86

serge-sans-paille updated this revision to Diff 239166. serge-sans-paille added a comment. - Harmonize esp/rsp usage as hinted by @craig.topper - Fix argument detection Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68720/new/ https://reviews.llvm.o

[PATCH] D72897: List implicit operator== after implicit destructors in a vtable.

peter.smith added a comment. It is also failing on the other 32-bit arm bots, example http://lab.llvm.org:8011/builders/clang-cmake-armv7-quick/builds/13052/steps/ninja%20check%201/logs/FAIL%3A%20Clang%3A%3Avirtual-compare.cpp Looking at the failure // CHECK-SAME: @_ZThn8_N1AD1Ev and the out

[PATCH] D72488: [clang-tidy] Add check for CERT-OOP57-CPP

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/docs/clang-tidy/checks/cert-oop57-cpp.rst:25 + Default is an empty string. + The check will detect the following functions: + `std::memcpy`, `memcpy`, `std::memmove`, `memmove`, `std::strcpy`, -

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

njames93 updated this revision to Diff 239167. njames93 marked 4 inline comments as done. njames93 added a comment. - Added not type template test case Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73052/new/ https://reviews.llvm.org/D73052 Files:

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

njames93 added inline comments. Comment at: clang-tools-extra/clang-tidy/utils/RenamerClangTidyCheck.cpp:282 +: DepMemberRef->getBaseType(); +CXXRecordDecl *Base = BaseType.getTypePtr()->getAsCXXRecordDecl(); +if (!Base) JonasToth

[PATCH] D72675: [Clang][Driver] Fix -ffast-math/-ffp-contract interaction

hfinkel added a comment. > I'm not sure whether this is deliberate (but it seems weird) or just a bug. I > can ask the GCC developers ... Please do. If there's a rationale, we should know. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72675/new/ https://reviews.llvm.org/D72675

[PATCH] D73056: [clangd] Fix DocumentOutline for concepts

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62012 tests passed, 0 failed and 783 were skipped. {icon question-circle color=gray} clang-tidy: unknown. {icon check-circle color=green} clang-format: pass. Build artifacts

[PATCH] D72557: Add pretty printers for llvm::PointerIntPair and llvm::PointerUnion.

csigg added a comment. Hi David. Thanks a lot for you review. I did another pass to switch to factory methods that catch exceptions without spewing common errors on the GDB console. Would you mind taking another quick peek? Thanks a lot for your patience! Repository: rG LLVM Github Monorepo

[PATCH] D72557: Add pretty printers for llvm::PointerIntPair and llvm::PointerUnion.

csigg updated this revision to Diff 239160. csigg added a comment. Herald added a subscriber: wuzish. Fix base. I'm so terrible at this. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72557/new/ https://reviews.llvm.org/D72557 Files: debuginfo-te

[PATCH] D73056: [clangd] Fix DocumentOutline for concepts

kadircet created this revision. kadircet added a reviewer: kbobyrev. Herald added subscribers: cfe-commits, usaxena95, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Fixes https://github.com/clangd/clangd/issues/256 Repository: rG LLVM Github Monorepo https://revie

[PATCH] D71512: [clang-format] Fix short block when braking after control statement

Bouska marked an inline comment as done. Bouska added inline comments. Comment at: clang/unittests/Format/FormatTest.cpp:662 "{\n" - " ff();\n" "}", MyDeveloperDay

[PATCH] D72675: [Clang][Driver] Fix -ffast-math/-ffp-contract interaction

uweigand added a comment. I've had a quick look at GCC, and it seems there's a couple of different issues. First of all, `-ffast-math` is a "collective" flag that has no separate meaning except setting a bunch of other flags. Currently, these flags are: `-fno-math-errno`, ` -funsafe-math-opti

[PATCH] D73007: [Sema] Avoid Wrange-loop-analysis false positives

aaronpuchert added a comment. In D73007#1829709 , @xbolva00 wrote: > Yes, but minimal fix is better for release branch, so @hans should merge it. I think `-Wall` shouldn't warn about reference types in a range-based for loop (unless it's the wrong type

[PATCH] D62445: [test] Fix plugin tests

mikhail.ramalho added a comment. Herald added a subscriber: Charusso. Hi, I just found this revision. Could you point out how can I run the CSA tests with Z3 as backend? We used to do `$ ninja check-clang-analyzer-z3`, however, it now fails and suggests: $ ninja check-clang-analyzer-z3 nin

[PATCH] D72488: [clang-tidy] Add check for CERT-OOP57-CPP

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62027 tests passed, 0 failed and 783 were skipped. {icon question-circle color=gray} clang-tidy: unknown. {icon times-circle color=red} clang-format: fail. Please format your changes with clang-format by runnin

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 61955 tests passed, 0 failed and 783 were skipped. {icon question-circle color=gray} clang-tidy: unknown. {icon times-circle color=red} clang-format: fail. Please format your changes with clang-format by runnin

[PATCH] D44609: [Clang-Format] New option BeforeLambdaBody to manage lambda line break inside function parameter call (in Allman style)

Wawha marked 2 inline comments as done. Wawha added a comment. Hello, @klimek will you have time to review this patch? The current patch integrate the last remarks and also modification due to inline lambda. Comment at: lib/Format/ContinuationIndenter.cpp:1179 +

[PATCH] D71512: [clang-format] Fix short block when braking after control statement

Bouska marked an inline comment as done. Bouska added inline comments. Comment at: clang/lib/Format/UnwrappedLineFormatter.cpp:344 - return MergedLines; -} // Don't merge block with left brace wrapped after ObjC special blocks MyDeveloperDay wrote:

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 61955 tests passed, 0 failed and 783 were skipped. {icon question-circle color=gray} clang-tidy: unknown. {icon times-circle color=red} clang-format: fail. Please format your changes with clang-format by runnin

[PATCH] D72488: [clang-tidy] Add check for CERT-OOP57-CPP

njames93 updated this revision to Diff 239153. njames93 added a comment. - rebase truck Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72488/new/ https://reviews.llvm.org/D72488 Files: clang-tools-extra/clang-tidy/cert/CERTTidyModule.cpp clang-

[PATCH] D72488: [clang-tidy] Add check for CERT-OOP57-CPP

This revision was automatically updated to reflect the committed changes. Closed by commit rGa42c3eb599cb: [clang-tidy] Add check for CERT-OOP57-CPP (authored by njames93). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72488/new/ https://reviews.llv

[clang-tools-extra] a42c3eb - [clang-tidy] Add check for CERT-OOP57-CPP

Author: Nathan Date: 2020-01-20T17:09:03Z New Revision: a42c3eb599cb3b83a07e6296d4ade213f1d74f0f URL: https://github.com/llvm/llvm-project/commit/a42c3eb599cb3b83a07e6296d4ade213f1d74f0f DIFF: https://github.com/llvm/llvm-project/commit/a42c3eb599cb3b83a07e6296d4ade213f1d74f0f.diff LOG: [clang

[PATCH] D72841: [RFC] Add support for pragma float_control, to control precision and exception behavior at the source level

mibintc marked 2 inline comments as done. mibintc added a comment. Added inline reply Comment at: clang/lib/CodeGen/CodeGenFunction.cpp:111 // the corresponding enum in the IR. -static llvm::fp::RoundingMode ToConstrainedRoundingMD( +llvm::fp::RoundingMode clang::ToConstrained

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

JonasToth added inline comments. Comment at: clang-tools-extra/clang-tidy/utils/RenamerClangTidyCheck.cpp:282 +: DepMemberRef->getBaseType(); +CXXRecordDecl *Base = BaseType.getTypePtr()->getAsCXXRecordDecl(); +if (!Base) can `Base

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

njames93 updated this revision to Diff 239150. njames93 added a comment. - replace auto when type isnt explicit Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73052/new/ https://reviews.llvm.org/D73052 Files: clang-tools-extra/clang-tidy/utils/Re

[PATCH] D73052: [clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved

njames93 created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. njames93 retitled this revision from "First Draft" to "[clang-tidy] RenamerClangTidy now renames dependent member expr when the member can be resolved". njames93 added reviewers: aaron.ballman,

[PATCH] D72932: [ARM] Follow AACPS standard for volatile bit-fields access width

ostannard added a comment. Why are you doing this in CodeGen, rather than adjusting the existing layout code in CGRecordLowering? Doing it this way will result in AdjustAAPCSBitfieldLValue being called for every access to the bitfield, rather than just once. This is probably more fragile too, b

[PATCH] D71698: [AArch64][SVE] Add intrinsic for non-faulting loads

kmclaughlin updated this revision to Diff 239144. kmclaughlin added a comment. - Some minor changes to performSignExtendInRegCombine to address comments from @sdesmalen CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71698/new/ https://reviews.llvm.org/D71698 Files: llvm/include/llvm/

[PATCH] D73007: [Sema] Avoid Wrange-loop-analysis false positives

xbolva00 added a subscriber: hans. xbolva00 added a comment. Yes, but minimal fix is better for release branch, so @hans should merge it. Handling your case probably needs more work and patches.. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73007/

[PATCH] D71566: New checks for fortified sprintf

aaron.ballman added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:745-746 +def warn_fortify_source_format_overflow : Warning< + "'%0' will always overflow; destination buffer has size %1," + " but format string expands to at least %2">, + InGrou

[clang] 3663563 - [OPENMP]Fix PR44578: crash in target construct with captured global.

Author: Alexey Bataev Date: 2020-01-20T11:10:17-05:00 New Revision: 366356361cf3abf97fbcfe9a0467b1ed1610401f URL: https://github.com/llvm/llvm-project/commit/366356361cf3abf97fbcfe9a0467b1ed1610401f DIFF: https://github.com/llvm/llvm-project/commit/366356361cf3abf97fbcfe9a0467b1ed1610401f.diff

[PATCH] D73007: [Sema] Avoid Wrange-loop-analysis false positives

aaronpuchert added a comment. As I wrote on the bug , I think we should only suppress one of the warnings in templates (and maybe always): :18:22: warning: loop variable '_' is always a copy because the range of type 'const Rng' does not return a r

patch via mailing list: Use getLocation() in too few/many arguments diagnostic

This small patch improves the diagnostics when calling a function with the wrong number of arguments. For example, given int foo(int a, int b); int bar() { return foo(1); } The existing compiler shows the error message and a note for "foo declared here" showing only the

[PATCH] D73026: clang-format: [JS] fix `??` opreator wrapping.

This revision was automatically updated to reflect the committed changes. Closed by commit rG4c9d6914453d: clang-format: [JS] fix `??` opreator wrapping. (authored by mprobst). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73026/new/ https://reviews

[clang] 4c9d691 - clang-format: [JS] fix `??` opreator wrapping.

Author: Martin Probst Date: 2020-01-20T17:07:14+01:00 New Revision: 4c9d6914453d970b7b8202b7efd7524b2f0a72ac URL: https://github.com/llvm/llvm-project/commit/4c9d6914453d970b7b8202b7efd7524b2f0a72ac DIFF: https://github.com/llvm/llvm-project/commit/4c9d6914453d970b7b8202b7efd7524b2f0a72ac.diff

[PATCH] D73035: [clangd] Avoid redundant testcases in rename unittest, NFC.

This revision was automatically updated to reflect the committed changes. Closed by commit rG3de9a5db629a: [clangd] Avoid redundant testcases in rename unittest, NFC. (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73035/new/ htt

[clang-tools-extra] 3de9a5d - [clangd] Avoid redundant testcases in rename unittest, NFC.

Author: Haojian Wu Date: 2020-01-20T17:01:05+01:00 New Revision: 3de9a5db629ac9d633576513d025a8f038a3cdec URL: https://github.com/llvm/llvm-project/commit/3de9a5db629ac9d633576513d025a8f038a3cdec DIFF: https://github.com/llvm/llvm-project/commit/3de9a5db629ac9d633576513d025a8f038a3cdec.diff LO

[PATCH] D71607: [clang-tidy] Add unsigned subtraction warning, with suggestion to convert to unsigned literals.

aaron.ballman added a comment. In D71607#1828055 , @sorenj wrote: > Okay, but as you can see the majority of my test cases are intentionally > false negatives `- -Wsign-conversion` triggers so often than many people > don't use it. Then we should be ad

[PATCH] D73007: [Sema] Avoid Wrange-loop-analysis false positives

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM! Do you think this should also be pushed onto the release branch? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73007/new/ ht

[PATCH] D72488: [clang-tidy] Add check for CERT-OOP57-CPP

aaron.ballman added a comment. Btw, do you need me to commit this on your behalf, or have you obtained your commit privileges? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72488/new/ https://reviews.llvm.org/D72488

[PATCH] D71512: [clang-format] Fix short block when braking after control statement

MyDeveloperDay requested changes to this revision. MyDeveloperDay added a comment. This revision now requires changes to proceed. In D71512#1815512 , @Bouska wrote: > Ping @MyDeveloperDay @klimek Comment at: clang/lib/Format/Unwrappe

[PATCH] D72755: [RISCV] Pass target-abi via module flag metadata

lenary accepted this revision. lenary added a comment. This revision is now accepted and ready to land. LGTM, Thank You! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72755/new/ https://reviews.llvm.org/D72755 ___

[PATCH] D71451: Support to emit extern variables debuginfo with "-fstandalone-debug"

Jac1494 updated this revision to Diff 239115. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71451/new/ https://reviews.llvm.org/D71451 Files: clang/include/clang/Basic/TargetInfo.h clang/lib/CodeGen/CGDebugInfo.cpp clang/test/CodeGen/debug-info-extern-variable-basic.c clang/test/

[PATCH] D72830: [ARM][MVE][Intrinsics] Take abs() of VMINNMAQ, VMAXNMAQ intrinsics' first arguments.

This revision was automatically updated to reflect the committed changes. Closed by commit rGb10a0eb04adf: [ARM][MVE][Intrinsics] Take abs() of VMINNMAQ, VMAXNMAQ intrinsics' first… (authored by MarkMurrayARM). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llv

[clang] b10a0eb - [ARM][MVE][Intrinsics] Take abs() of VMINNMAQ, VMAXNMAQ intrinsics' first arguments.

Author: Mark Murray Date: 2020-01-20T14:33:26Z New Revision: b10a0eb04adfc4186cc6198cf8231358b2b04d89 URL: https://github.com/llvm/llvm-project/commit/b10a0eb04adfc4186cc6198cf8231358b2b04d89 DIFF: https://github.com/llvm/llvm-project/commit/b10a0eb04adfc4186cc6198cf8231358b2b04d89.diff LOG: [

[PATCH] D72830: [ARM][MVE][Intrinsics] Take abs() of VMINNMAQ, VMAXNMAQ intrinsics' first arguments.

MarkMurrayARM updated this revision to Diff 239112. MarkMurrayARM added a comment. Rebase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72830/new/ https://reviews.llvm.org/D72830 Files: clang/include/clang/Basic/arm_mve.td clang/test/CodeGen/

[PATCH] D71001: [clang-tidy] New check: bugprone-misplaced-pointer-arithmetic-in-alloc

baloghadamsoftware updated this revision to Diff 239103. baloghadamsoftware added a comment. Updated according to the comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71001/new/ https://reviews.llvm.org/D71001 Files: clang-tools-extra/clang-tidy/bugprone/BugproneTidyModule.cpp

[PATCH] D71001: [clang-tidy] New check: bugprone-misplaced-pointer-arithmetic-in-alloc

baloghadamsoftware marked 5 inline comments as done. baloghadamsoftware added a comment. Hello, Thank you for your useful comments. I think I fixed them all now. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71001/new/ https://reviews.llvm.org/D71001

[PATCH] D72934: [ARM,MVE] Support immediate vbicq,vorrq,vmvnq intrinsics.

dmgreen added a comment. In D72934#1829387 , @simon_tatham wrote: > In D72934#1829331 , @dmgreen wrote: > > > What is the reason that this can't be lowered in tablegen, in the same way > > as the VMOVimm's are? >

[PATCH] D72675: [Clang][Driver] Fix -ffast-math/-ffp-contract interaction

spatel added a comment. In D72675#1827893 , @wristow wrote: > How to handle this seems like an implementation question. The code here is > just deciding whether or not we intend to pass "-ffast-math" to cc1 (it isn't > directly defining `__FAST_MATH__`)

[PATCH] D72934: [ARM,MVE] Support immediate vbicq,vorrq,vmvnq intrinsics.

simon_tatham marked 2 inline comments as done. simon_tatham added a comment. In D72934#1829331 , @dmgreen wrote: > What is the reason that this can't be lowered in tablegen, in the same way as > the VMOVimm's are? In NEON, immediate VBIC is represented

[PATCH] D71451: Support to emit extern variables debuginfo with "-fstandalone-debug"

Jac1494 updated this revision to Diff 239089. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71451/new/ https://reviews.llvm.org/D71451 Files: clang/include/clang/Basic/TargetInfo.h clang/lib/CodeGen/CGDebugInfo.cpp Index: clang/lib/CodeGen/CGDebugInfo.cpp ==

[PATCH] D73037: Add a way to set traversal mode in clang-query

steveire created this revision. steveire added a reviewer: aaron.ballman. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D73037 Files: clang-tools-extra/clang-query/Query.cpp clang-tools-extra/clang-query/

[PATCH] D73035: [clangd] Avoid redundant testcases in rename unittest, NFC.

merge_guards_bot added a comment. {icon check-circle color=green} Unit tests: pass. 62009 tests passed, 0 failed and 783 were skipped. {icon question-circle color=gray} clang-tidy: unknown. {icon check-circle color=green} clang-format: pass. Build artifacts

  1   2   >