[PATCH] D77229: [Analyzer] Avoid handling of LazyCompundVals in IteratorModeling

baloghadamsoftware added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/Iterator.cpp:330-336 +SVal getReturnIterator(const CallEvent &Call) { + Optional RetValUnderConstr = Call.getReturnValueUnderConstruction(); + if (RetValUnderConstr.hasValue()) +return *

[PATCH] D77229: [Analyzer] Avoid handling of LazyCompundVals in IteratorModeling

baloghadamsoftware added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/Iterator.cpp:330-336 +SVal getReturnIterator(const CallEvent &Call) { + Optional RetValUnderConstr = Call.getReturnValueUnderConstruction(); + if (RetValUnderConstr.hasValue()) +return *

[clang] d7eb917 - [PowerPC] Implementation of 128-bit Binary Vector Mod and Sign Extend builtins

Author: Albion Fung Date: 2020-09-23T01:18:14-05:00 New Revision: d7eb917a7cb793f49e16841fc24826b988dd5c8f URL: https://github.com/llvm/llvm-project/commit/d7eb917a7cb793f49e16841fc24826b988dd5c8f DIFF: https://github.com/llvm/llvm-project/commit/d7eb917a7cb793f49e16841fc24826b988dd5c8f.diff L

[PATCH] D88100: [analyzer][StdLibraryFunctionsChecker] Separate the signature from the summaries

Szelethus accepted this revision. Szelethus added a comment. This revision is now accepted and ready to land. Yup, we've talked about this before. This is indeed a better interface design. LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88100/n

[PATCH] D88092: [analyzer][StdLibraryFunctionsChecker] Fix getline/getdelim signatures

Szelethus accepted this revision. Szelethus added a comment. This revision is now accepted and ready to land. A joy of reviewing C++ code is that you get to marvel in all the great things the language has, without having to pull fistfuls of hair out to get get to that point. These patches are al

[PATCH] D88133: [Analyzer][WebKit] Use tri-state types for relevant predicates

jkorous added a comment. Created this for eventual post-commit review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88133/new/ https://reviews.llvm.org/D88133 ___ cfe-commits mailing list cfe-commits@li

[PATCH] D88133: [Analyzer][WebKit] Use tri-state types for relevant predicates

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rG47e6851423fd: [Analyzer][WebKit] Use tri-state types for relevant predicates (authored by jkorous). Herald added a project

[clang] 47e6851 - [Analyzer][WebKit] Use tri-state types for relevant predicates

Author: Jan Korous Date: 2020-09-22T21:57:24-07:00 New Revision: 47e6851423fd32f0685a643236ad946e23ab14ff URL: https://github.com/llvm/llvm-project/commit/47e6851423fd32f0685a643236ad946e23ab14ff DIFF: https://github.com/llvm/llvm-project/commit/47e6851423fd32f0685a643236ad946e23ab14ff.diff LO

[PATCH] D87921: Fix -funique-internal-linkage-names to work with -O2 and new pass manager

nemanjai added a comment. It turns out that the culprit for the PPC bot failures is actually https://reviews.llvm.org/rG144e57fc9535 But this just took a while to manifest. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87921/new/ https://reviews.l

[PATCH] D87921: Fix -funique-internal-linkage-names to work with -O2 and new pass manager

MaskRay added a comment. When you recommit, consider adding the header to the description. The original message started with "The wrong placement of add pass with optimizations led to -funique-internal-linkage-names being disabled." The header was missing. Repository: rG LLVM Github Monorepo

[PATCH] D87921: Fix -funique-internal-linkage-names to work with -O2 and new pass manager

nemanjai added a comment. In D87921#2288686 , @morehouse wrote: > The revert did not fix the PPC bots. I suspect there is some kind of > resource issue from the logs: > > msgget:: No space left on device > sysmsg.c.tmp: > /home/buildbots/ppc64le-cla

[PATCH] D87953: [xray] Function coverage groups

MaskRay added inline comments. Comment at: clang/lib/Frontend/CompilerInvocation.cpp:1136 + if (Opts.XRayTotalFunctionGroups < 1) { +const Arg *A = Args.getLastArg(OPT_fxray_function_groups); +Diags.Report(diag::err_drv_invalid_value) Errors here are not

[PATCH] D88130: [PPC] [AIX] Implement calling convention IR for C99 complex types on AIX

cebowleratibm created this revision. cebowleratibm added reviewers: ZarkoCA, daltenty, sfertile. Herald added subscribers: cfe-commits, nemanjai. Herald added a project: clang. cebowleratibm requested review of this revision. Builds on D88105 to add AIX calling c

[PATCH] D88105: [NFC] [PPC] Add PowerPC expected IR tests for C99 complex

cebowleratibm updated this revision to Diff 293625. cebowleratibm marked an inline comment as done. cebowleratibm added a comment. Added ppc64le target. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88105/new/ https://reviews.llvm.org/D88105 Files: clang/test/CodeGen/powerpc-c99compl

[PATCH] D88003: Fix typos in ASTMatchers.h

YangZhihui added a comment. In D88003#2287580 , @aaron.ballman wrote: > LGTM! I don't have commit access maybe you can help me commit it Thanks : ) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88003/new/ https://reviews.llvm.org/D88003

[PATCH] D87808: [DebugInfo] Fix bug in constructor homing where it would use ctor homing when a class only has copy/move constructors

dblaikie added inline comments. Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:2292-2300 + bool hasCtor = false; + for (const auto *Ctor : RD->ctors()) { if (Ctor->isTrivial() && !Ctor->isCopyOrMoveConstructor()) return false; +if (!Ctor->isCopyOrMoveConstructor(

[PATCH] D87974: [Builtin] Add __builtin_zero_non_value_bits.

zoecarver updated this revision to Diff 293606. zoecarver edited the summary of this revision. zoecarver added a comment. - Add more test cases. - Fix typo. - Add codegen tests. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87974/new/ https://revie

[clang] 15d94a7 - Revert "Canonicalize declaration pointers when forming APValues."

Author: Leonard Chan Date: 2020-09-22T17:40:53-07:00 New Revision: 15d94a7d0f8f0d6b3b5308fff51b286957e45650 URL: https://github.com/llvm/llvm-project/commit/15d94a7d0f8f0d6b3b5308fff51b286957e45650 DIFF: https://github.com/llvm/llvm-project/commit/15d94a7d0f8f0d6b3b5308fff51b286957e45650.diff

[PATCH] D54943: WIP [clang-tidy] implement const-transformation for cppcoreguidelines-const-correctness

0x8000- added a comment. Master branch has too many false positives for tidy - would it be possible to create a branch that contains this patch set on top of llvm-11.0-rc3? I would then add this to our internal CI. For the legacy code (see previous reports) we found quite a few false posit

[clang] 1009229 - [Clang] Fix a typo in implicit-int-float-conversion.c

Author: Yuanfang Chen Date: 2020-09-22T16:51:23-07:00 New Revision: 10092291d7a770fee9eec8d8c7f60aeca9a8d7fb URL: https://github.com/llvm/llvm-project/commit/10092291d7a770fee9eec8d8c7f60aeca9a8d7fb DIFF: https://github.com/llvm/llvm-project/commit/10092291d7a770fee9eec8d8c7f60aeca9a8d7fb.diff

[PATCH] D88121: [X86] Add a memory clobber to the bittest intrinsic inline asm. Get default clobbers from the target

craig.topper created this revision. craig.topper added reviewers: RKSimon, spatel, rnk, echristo. craig.topper requested review of this revision. I believe the inline asm emitted here should have a memory clobber since it writes to memory. It was also missing the dirflag clobber that we use by d

[PATCH] D80344: [Windows SEH]: HARDWARE EXCEPTION HANDLING (MSVC -EHa) - Part 1

tentzen added a comment. In D80344#2286838 , @rjmccall wrote: > In D80344#228 , @tentzen wrote: > >> Thank you for prompt reply again. >> >>> [rjmccall] And I agree with him that the potential benefits are >>>

[PATCH] D78075: [Clang][OpenMP] Added support for nowait target in CodeGen

tianshilei1992 added a comment. @jdoerfert Ready to be reviewed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78075/new/ https://reviews.llvm.org/D78075 ___ cfe-commits mailing list cfe-commits@lists.ll

[PATCH] D87808: [DebugInfo] Fix bug in constructor homing where it would use ctor homing when a class only has copy/move constructors

akhuang updated this revision to Diff 293579. akhuang added a comment. Update ctor homing check, and add some test cases. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87808/new/ https://reviews.llvm.org/D87808 Files: clang/lib/CodeGen/CGDebugIn

[PATCH] D87451: add new clang option -mno-xcoff-visibility

daltenty added inline comments. Comment at: llvm/include/llvm/Target/TargetMachine.h:265 + /// corresponding to -mno-xcoff-visibility. + bool getNoXCOFFVisibility() const { return Options.NoXCOFFVisibility; } + This seems like it needs the corresponding comand-

[PATCH] D88114: [clang]Test ensuring -fembed-bitcode passed to cc1 captures pre-opt bitcode.

steven_wu added a comment. Ok, I guess we are on the same page. The idea sounds fine to me. I would suggest just check that the output matches the input file as much as possible, rather than just check a label and a call instruction. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST AC

[PATCH] D88114: [clang]Test ensuring -fembed-bitcode passed to cc1 captures pre-opt bitcode.

mtrofin added a comment. In D88114#2288749 , @steven_wu wrote: > In D88114#2288737 , @mtrofin wrote: > >> In D88114#2288732 , @steven_wu >> wrote: >> >>> I am not sure what

[PATCH] D78075: [Clang][OpenMP] Added support for nowait target in CodeGen

tianshilei1992 updated this revision to Diff 293577. tianshilei1992 added a comment. Rebased the source code and refined `declare_mapper_codegen.cpp` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78075/new/ https://reviews.llvm.org/D78075 Files:

[PATCH] D87808: [DebugInfo] Fix bug in constructor homing where it would use ctor homing when a class only has copy/move constructors

akhuang added inline comments. Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:2292-2300 + bool hasCtor = false; + for (const auto *Ctor : RD->ctors()) { if (Ctor->isTrivial() && !Ctor->isCopyOrMoveConstructor()) return false; +if (!Ctor->isCopyOrMoveConstructor()

[PATCH] D87652: Sema: add support for `__attribute__((__swift_newtype__))`

compnerd updated this revision to Diff 293574. compnerd marked 6 inline comments as done. compnerd added a comment. Update for feedback from @aaron.ballman The `ParseAttrCommonArgs` refactoring still needs to be looked at, but this should address the feedback, and doesn't need to be predicated o

[PATCH] D87451: add new clang option -mno-xcoff-visibility

DiggerLin updated this revision to Diff 293571. DiggerLin added a comment. Herald added subscribers: llvm-commits, kbarton, hiraditya, nemanjai. Herald added a project: LLVM. address comment Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87451/new/

[PATCH] D88103: [JSON] Add error reporting facility, used in fromJSON and ObjectMapper.

wallace added a comment. I like this a lot!! Amazing work. I think it'll be useful for my Trace patch. I'll do a corresponding refactor using probably this new functionality. I find the diff straightforward to follow. Unless anyone has objections, I'm fine with the patch at it is. Repository:

[PATCH] D88114: [clang]Test ensuring -fembed-bitcode passed to cc1 captures pre-opt bitcode.

steven_wu added a comment. In D88114#2288737 , @mtrofin wrote: > In D88114#2288732 , @steven_wu wrote: > >> I am not sure what exactly is expected here. What is your definition for >> pre-optimized bitcode and how

[PATCH] D88114: [clang]Test ensuring -fembed-bitcode passed to cc1 captures pre-opt bitcode.

mtrofin added a comment. In D88114#2288732 , @steven_wu wrote: > I am not sure what exactly is expected here. What is your definition for > pre-optimized bitcode and how your test case ensures that? Can you explain a > bit more for context? Pre-optimize

[PATCH] D88114: [clang]Test ensuring -fembed-bitcode passed to cc1 captures pre-opt bitcode.

steven_wu added a comment. I am not sure what exactly is expected here. What is your definition for pre-optimized bitcode and how your test case ensures that? Can you explain a bit more for context? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88

[PATCH] D88115: [CUDA][HIP] Fix static device var used by host code only

yaxunl created this revision. yaxunl added a reviewer: tra. yaxunl requested review of this revision. A static device variable may be accessed in host code through cudaMemCpyFromSymbol etc. Currently clang does not emit the static device variable if it is only referenced by host code, which causes

[PATCH] D86790: [FE] Use preferred alignment instead of ABI alignment for complete object when applicable

Xiangling_L updated this revision to Diff 293561. Xiangling_L marked 16 inline comments as done. Xiangling_L added a comment. Addressed the comments; CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86790/new/ https://reviews.llvm.org/D86790 Files: clang/include/clang/AST/ASTContext.h

[PATCH] D88084: [clang-format] Changed default styles BraceWrappping bool table to directly use variables

MarcusJohnson91 added a comment. In D88084#2287450 , @MyDeveloperDay wrote: > I noticed the pre-merge tests failed! Yeah I just noticed that too, not sure what's up but I'll check into it, and yeah that's a good idea about initializing some of these dup

[PATCH] D87956: [WIP][IR] add fn attr for no_stack_protector; prevent inlining ssp into no-ssp

nickdesaulniers added a comment. I obviously have a few todos left to resolve, but CC'ing reviewers now for general feedback. One thing I'm unsure of; it felt curious to me that nothing was enforcing that these different levels of stack protection were mutually exclusive. I've added that here

[PATCH] D87953: [xray] Function coverage groups

ianlevesque added a comment. This is ready for another review, I think I addressed everything. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87953/new/ https://reviews.llvm.org/D87953 ___ cfe-commits mai

[PATCH] D87921: Fix -funique-internal-linkage-names to work with -O2 and new pass manager

morehouse added a comment. The revert did not fix the PPC bots. I suspect there is some kind of resource issue from the logs: msgget:: No space left on device sysmsg.c.tmp: /home/buildbots/ppc64le-clang-lnt-test/clang-ppc64le-lnt/llvm/compiler-rt/test/sanitizer_common/TestCases/Linux/sysms

[PATCH] D87956: [WIP][IR] add fn attr for no_stack_protector; prevent inlining ssp into no-ssp

nickdesaulniers updated this revision to Diff 293559. nickdesaulniers added a comment. Herald added a reviewer: whitequark. Herald added a reviewer: aaron.ballman. - add tests, make fn attrs mutually exclusive Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llv

[PATCH] D88114: [clang]Test ensuring -fembed-bitcode passed to cc1 captures pre-opt bitcode.

mtrofin created this revision. mtrofin added reviewers: steven_wu, tejohnson. Herald added a reviewer: alexshap. Herald added subscribers: cfe-commits, dexonsmith. Herald added a project: clang. mtrofin requested review of this revision. This is important to not regress because it allows us to cap

[PATCH] D87808: [DebugInfo] Fix bug in constructor homing where it would use ctor homing when a class only has copy/move constructors

rsmith added a comment. In D87808#2288186 , @dblaikie wrote: > In D87808#2286664 , @rsmith wrote: > >> But I think this code should give the same outcome either way, because a >> class with any constructor other th

[PATCH] D86694: [scudo] Allow -fsanitize=scudo on Linux and Windows (WIP, don't land as is)

aganea added a comment. I'm also in favor, I think this is good direction ahead. It'd be nice if following issues were fixed -- in subsequent patches if you wish: - Stage1 `ninja check-scudo` fails many tests for me, see F13037612: errors.txt . - Stage2 `ninj

[PATCH] D77229: [Analyzer] Avoid handling of LazyCompundVals in IteratorModeling

NoQ added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/Iterator.cpp:330-336 +SVal getReturnIterator(const CallEvent &Call) { + Optional RetValUnderConstr = Call.getReturnValueUnderConstruction(); + if (RetValUnderConstr.hasValue()) +return *RetValUnderCons

[clang] cf11238 - [ThinLTO] Option to bypass function importing.

Author: Mircea Trofin Date: 2020-09-22T13:12:11-07:00 New Revision: cf112382ddd0d717edf0b8a3e5b061de925258e9 URL: https://github.com/llvm/llvm-project/commit/cf112382ddd0d717edf0b8a3e5b061de925258e9 DIFF: https://github.com/llvm/llvm-project/commit/cf112382ddd0d717edf0b8a3e5b061de925258e9.diff

[PATCH] D87949: [ThinLTO] Option to bypass function importing.

This revision was automatically updated to reflect the committed changes. Closed by commit rGcf112382ddd0: [ThinLTO] Option to bypass function importing. (authored by mtrofin). Changed prior to commit: https://reviews.llvm.org/D87949?vs=293506&id=293555#toc Repository: rG LLVM Github Monorep

[PATCH] D87949: [ThinLTO] Option to bypass function importing.

tejohnson accepted this revision. tejohnson added a comment. This revision is now accepted and ready to land. lgtm with comment typo fix Comment at: llvm/include/llvm/LTO/LTOBackend.h:59 + +/// Distributed ThinlTO: load the referenced modules, keeping their buffers +/// alive i

[PATCH] D87921: Fix -funique-internal-linkage-names to work with -O2 and new pass manager

MaskRay added a comment. In D87921#2288395 , @Conanap wrote: > It looks like this commit is causing a few failures on nearly all PPC bots > and a sanitizer bot; would it be possible to revert this commit for now until > the issue is resolved? It would b

[clang] b89059a - Revert "The wrong placement of add pass with optimizations led to -funique-internal-linkage-names being disabled."

Author: Sriraman Tallam Date: 2020-09-22T12:32:43-07:00 New Revision: b89059a31347dd09b55a96b99b3dbe38d7749908 URL: https://github.com/llvm/llvm-project/commit/b89059a31347dd09b55a96b99b3dbe38d7749908 DIFF: https://github.com/llvm/llvm-project/commit/b89059a31347dd09b55a96b99b3dbe38d7749908.dif

[PATCH] D87946: [OpenMP] Add Location Fields to Libomptarget Runtime for Debugging

jhuber6 updated this revision to Diff 293550. jhuber6 added a comment. Updated runtime library to have legacy calls into the new functions with source location information. Because the library interface requires C function naming this required adding a suffix to all the functions clang generates

Re: [PATCH] D87921: Fix -funique-internal-linkage-names to work with -O2 and new pass manager

On Tue, Sep 22, 2020 at 11:28 AM Albion Fung via Phabricator wrote: > > Conanap added a comment. > > It looks like this commit is causing a few failures on nearly all PPC bots > and a sanitizer bot; would it be possible to revert this commit for now until > the issue is resolved? Ya sure, I can

[PATCH] D84637: [AST] Enhance the const expression evaluator to support error-dependent exprs.

hokein added a comment. @rsmith this patch should be ready for another round of review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84637/new/ https://reviews.llvm.org/D84637 ___ cfe-commits mailing li

[PATCH] D84637: [AST] Enhance the const expression evaluator to support error-dependent exprs.

hokein updated this revision to Diff 293538. hokein added a comment. rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84637/new/ https://reviews.llvm.org/D84637 Files: clang/lib/AST/ExprConstant.cpp clang/test/SemaCXX/constexpr-function-re

[PATCH] D83296: [clang-format] Add a MacroExpander.

sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. Ship it! Comment at: clang/lib/Format/FormatToken.h:177 +/// EndOfExpansion: 0 0 0 21 0 1 +struct MacroContext { + MacroContext(MacroRole Role) : Role(Rol

[PATCH] D86895: [Modules] Add stats to measure performance of building and loading modules.

vsapsai added a comment. In D86895#2288262 , @tschuett wrote: > Where is the test? `ALWAYS_ENABLED_STATISTIC` has its own unit tests. And tests that use added stats are in another revision in the stack D86896 . Reposit

[PATCH] D87921: Fix -funique-internal-linkage-names to work with -O2 and new pass manager

Conanap added a comment. It looks like this commit is causing a few failures on nearly all PPC bots and a sanitizer bot; would it be possible to revert this commit for now until the issue is resolved? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D

[PATCH] D78075: [WIP][Clang][OpenMP] Added support for nowait target in CodeGen

tianshilei1992 updated this revision to Diff 293537. tianshilei1992 added a comment. Refined the case `declare_mapper_codegen.cpp` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78075/new/ https://reviews.llvm.org/D78075 Files: clang/lib/CodeGen/

[clang] 16ca711 - [clang] Fix a typo-correction crash

Author: Haojian Wu Date: 2020-09-22T20:21:21+02:00 New Revision: 16ca711803300bd966acf8759876a1ccd478c616 URL: https://github.com/llvm/llvm-project/commit/16ca711803300bd966acf8759876a1ccd478c616 DIFF: https://github.com/llvm/llvm-project/commit/16ca711803300bd966acf8759876a1ccd478c616.diff LO

[PATCH] D87815: [clang] Fix a typo-correction crash

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG16ca71180330: [clang] Fix a typo-correction crash (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://r

[PATCH] D87921: Fix -funique-internal-linkage-names to work with -O2 and new pass manager

saghir added a comment. In D87921#2288096 , @morehouse wrote: > Thanks for looking. Indeed, it looks like an issue with the disk being full > on the bot. Hi, I checked the disk is not full on the bot. I am not sure what is going on here but its definit

[PATCH] D83004: [UpdateCCTestChecks] Include generated functions if asked

dmajor added a comment. The expensive-checks bots have been red for several days. Could you please take a look or revert? riscv_generated_funcs.test: *** Bad machine code: MBB exits via unconditional fall-through but ends with a barrier instruction! *** - function:check_boundaries -

[clang] 8a64689 - [Analyzer][WebKit] UncountedLocalVarsChecker

Author: Jan Korous Date: 2020-09-22T11:05:04-07:00 New Revision: 8a64689e264ce039e4fb0a09c3e136a1c8451838 URL: https://github.com/llvm/llvm-project/commit/8a64689e264ce039e4fb0a09c3e136a1c8451838 DIFF: https://github.com/llvm/llvm-project/commit/8a64689e264ce039e4fb0a09c3e136a1c8451838.diff LO

[PATCH] D87949: [ThinLTO] Option to bypass function importing.

mtrofin updated this revision to Diff 293506. mtrofin added a comment. clang tidy Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87949/new/ https://reviews.llvm.org/D87949 Files: clang/include/clang/CodeGen/BackendUtil.h clang/lib/CodeGen/Backe

[PATCH] D83154: clang: Add -fcoverage-prefix-map

keith added a comment. Should we wait for that one to land and then pick this one up? Otherwise any thoughts on the outstanding discussion? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83154/new/ https://reviews.llvm.org/D83154 _

[PATCH] D86895: [Modules] Add stats to measure performance of building and loading modules.

tschuett added a comment. Where is the test? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86895/new/ https://reviews.llvm.org/D86895 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists

[PATCH] D86895: [Modules] Add stats to measure performance of building and loading modules.

bruno accepted this revision. bruno added a comment. This looks pretty useful, thanks for adding this Volodymyr. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86895/new/ https://reviews.llvm.org/D86895 ___

[PATCH] D78075: [WIP][Clang][OpenMP] Added support for nowait target in CodeGen

tianshilei1992 updated this revision to Diff 293505. tianshilei1992 added a comment. Fixed the case `target_parallel_for_codegen.cpp` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78075/new/ https://reviews.llvm.org/D78075 Files: clang/lib/CodeG

[PATCH] D86964: [ASTMatchers] Avoid recursion in ancestor matching to save stack space.

This revision was automatically updated to reflect the committed changes. sammccall marked an inline comment as done. Closed by commit rG625761825620: [ASTMatchers] Avoid recursion in ancestor matching to save stack space. (authored by sammccall). Changed prior to commit: https://reviews.llvm.o

[clang] 6257618 - [ASTMatchers] Avoid recursion in ancestor matching to save stack space.

Author: Sam McCall Date: 2020-09-22T19:43:41+02:00 New Revision: 625761825620f19a44c7a1482ce05d678a1b0deb URL: https://github.com/llvm/llvm-project/commit/625761825620f19a44c7a1482ce05d678a1b0deb DIFF: https://github.com/llvm/llvm-project/commit/625761825620f19a44c7a1482ce05d678a1b0deb.diff LO

[PATCH] D88105: [NFC] [PPC] Add PowerPC expected IR tests for C99 complex

ZarkoCA added inline comments. Comment at: clang/test/CodeGen/powerpc-c99complex.c:1 +// RUN: %clang_cc1 -triple powerpc64-unknown-linux -emit-llvm %s -o - | FileCheck %s --check-prefixes=PPC64LNX +// RUN: %clang_cc1 -triple powerpc-unknown-linux -emit-llvm %s -o - | FileCheck

[PATCH] D86964: [ASTMatchers] Avoid recursion in ancestor matching to save stack space.

sammccall marked 3 inline comments as done. sammccall added inline comments. Comment at: clang/lib/ASTMatchers/ASTMatchFinder.cpp:705 + // Unlike matchesAnyAncestorOf there's no memoization: it doesn't save much. + bool matchesParentOf(const DynTypedNode &Node, const DynTypedMa

[PATCH] D87989: [Flang][Driver] Add InputOutputTest frontend action with new -test-IO flag

awarzynski added a comment. @CarolineConcatto , thank you for this patch! It implements some really important functionality and IMO the overall structure is solid. I've left quite a few comments, but mostly nits and suggestions for more detailed comments. There's a few, but it's a relatively bi

[PATCH] D87808: [DebugInfo] Fix bug in constructor homing where it would use ctor homing when a class only has copy/move constructors

dblaikie added a comment. In D87808#2286664 , @rsmith wrote: > In D87808#2280197 , @dblaikie wrote: > >> @rsmith What's the deal with these anonymous structs/unions? Why do they >> have copy/move constructors (are

[clang] c3c08bf - [SyntaxTree] Test the List API

Author: Eduardo Caldas Date: 2020-09-22T17:07:41Z New Revision: c3c08bfdfd6244e0429753ee56df39c90187d772 URL: https://github.com/llvm/llvm-project/commit/c3c08bfdfd6244e0429753ee56df39c90187d772 DIFF: https://github.com/llvm/llvm-project/commit/c3c08bfdfd6244e0429753ee56df39c90187d772.diff LOG

[PATCH] D87839: [SyntaxTree] Test the List API

This revision was automatically updated to reflect the committed changes. Closed by commit rGc3c08bfdfd62: [SyntaxTree] Test the List API (authored by eduucaldas). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87839/new/ https://reviews.llvm.org/D87

[PATCH] D77229: [Analyzer] Avoid handling of LazyCompundVals in IteratorModeling

baloghadamsoftware marked an inline comment as done. baloghadamsoftware added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/Iterator.cpp:330-336 +SVal getReturnIterator(const CallEvent &Call) { + Optional RetValUnderConstr = Call.getReturnValueUnderConstruction(

[PATCH] D87588: [ASTMatchers] extract public matchers from const-analysis into own patch

JonasToth added a comment. In D87588#2278948 , @ro wrote: > As described in D87825 , this patch broke > `Debug` builds on `sparcv9-sun-solaris2.11` and `amd64-pc-solaris2.11`. Thank you for reporting this issue. I it is f

[PATCH] D54222: [clang-tidy] Add a check to detect returning static locals in public headers

JonasToth added a comment. In D54222#2281642 , @jranieri-grammatech wrote: > @JonasToth It looks like there are some outstanding review comments that need > to be addressed. I've gotten some time allocated to work on this next week. Great! Repository:

[PATCH] D88105: [NFC] [PPC] Add PowerPC expected IR tests for C99 complex

cebowleratibm created this revision. cebowleratibm added reviewers: daltenty, ZarkoCA, sfertile. Herald added subscribers: cfe-commits, shchenz, nemanjai. Herald added a project: clang. cebowleratibm requested review of this revision. Adding this test so that I can extend it in a follow on patch w

[PATCH] D88106: [SyntaxTree] Provide iterator-like functions for Lists

eduucaldas created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. eduucaldas requested review of this revision. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D88106 Files: clang/include/clang/Tooling/Syntax/Tree.h clang/lib/Tooling/Synt

[PATCH] D87671: [PowerPC] Implement Vector String Isolate Builtins in Clang/LLVM

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG079757b551f3: [PowerPC] Implement Vector String Isolate Builtins in Clang/LLVM (authored by amyk). Repository: rG LLVM Github Monorepo CHANGES SI

[clang] b314705 - [PowerPC] Implement the 128-bit Vector Divide Extended Builtins in Clang/LLVM

Author: Amy Kwan Date: 2020-09-22T11:31:44-05:00 New Revision: b3147058dec7d42ae0284d6e6edf25eb762c8b89 URL: https://github.com/llvm/llvm-project/commit/b3147058dec7d42ae0284d6e6edf25eb762c8b89 DIFF: https://github.com/llvm/llvm-project/commit/b3147058dec7d42ae0284d6e6edf25eb762c8b89.diff LOG:

[PATCH] D87729: [PowerPC] Implement the 128-bit Vector Divide Extended Builtins in Clang/LLVM

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGb3147058dec7: [PowerPC] Implement the 128-bit Vector Divide Extended Builtins in Clang/LLVM (authored by amyk). Repository: rG LLVM Github Monorep

[PATCH] D87921: Fix -funique-internal-linkage-names to work with -O2 and new pass manager

morehouse added a comment. Thanks for looking. Indeed, it looks like an issue with the disk being full on the bot. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87921/new/ https://reviews.llvm.org/D87921 _

[clang] 079757b - [PowerPC] Implement Vector String Isolate Builtins in Clang/LLVM

Author: Amy Kwan Date: 2020-09-22T11:31:44-05:00 New Revision: 079757b551f3ab5218af7344a7ab3c79976ec478 URL: https://github.com/llvm/llvm-project/commit/079757b551f3ab5218af7344a7ab3c79976ec478 DIFF: https://github.com/llvm/llvm-project/commit/079757b551f3ab5218af7344a7ab3c79976ec478.diff LOG:

Re: [PATCH] D87921: Fix -funique-internal-linkage-names to work with -O2 and new pass manager

On Tue, Sep 22, 2020 at 8:34 AM Fāng-ruì Sòng wrote: > > FWIW I tested check-msan in a -DCMAKE_BUILD_TYPE=Release build on a > powerpc64le machine. All tests passed. I cannot connect the failure to > the clang patch. Thanks for doing this! Much appreciated! > > On Mon, Sep 21, 2020 at 10:02 PM S

[PATCH] D87839: [SyntaxTree] Test the List API

eduucaldas marked an inline comment as done. eduucaldas added inline comments. Comment at: clang/include/clang/Tooling/Syntax/Tree.h:224 + /// "a, b c" <=> [("a", ","), ("b", nul), ("c", nul)] + /// "a, b,"<=> [("a", ","), ("b", ","), (nul, nul)] /// g

[PATCH] D87839: [SyntaxTree] Test the List API

eduucaldas updated this revision to Diff 293490. eduucaldas added a comment. . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87839/new/ https://reviews.llvm.org/D87839 Files: clang/include/clang/Tooling/Syntax/Tree.h clang/unittests/Tooling/Sy

[PATCH] D87839: [SyntaxTree] Test the List API

eduucaldas updated this revision to Diff 293489. eduucaldas marked 3 inline comments as done. eduucaldas added a comment. Answer code review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87839/new/ https://reviews.llvm.org/D87839 Files: clang/i

[PATCH] D88103: [JSON] Add error reporting facility, used in fromJSON and ObjectMapper.

sammccall created this revision. sammccall added reviewers: clayborg, kadircet, wallace. Herald added subscribers: llvm-commits, cfe-commits, usaxena95, arphaman, mgrang, hiraditya. Herald added projects: clang, LLVM. sammccall requested review of this revision. Herald added a subscriber: ilya-bir

[PATCH] D86819: [PowerPC][Power10] Implementation of 128-bit Binary Vector Rotate builtins

Conanap updated this revision to Diff 293474. Conanap marked 2 inline comments as done. Conanap added a comment. Changed implementation for vrlqnm as per Nemanja CHANGES SINCE LAST ACTION https://reviews.llvm.org/D86819/new/ https://reviews.llvm.org/D86819 Files: clang/include/clang/Basic/

[clang] 9bb5ecf - Sema: introduce `__attribute__((__swift_name__))`

Author: Saleem Abdulrasool Date: 2020-09-22T15:32:23Z New Revision: 9bb5ecf1f760e1b1fe4697189e4db99100baffad URL: https://github.com/llvm/llvm-project/commit/9bb5ecf1f760e1b1fe4697189e4db99100baffad DIFF: https://github.com/llvm/llvm-project/commit/9bb5ecf1f760e1b1fe4697189e4db99100baffad.diff

[PATCH] D87534: Sema: introduce `__attribute__((__swift_name__))`

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. compnerd marked 4 inline comments as done. Closed by commit rG9bb5ecf1f760: Sema: introduce `__attribute__((__swift_name__))` (authored by compnerd). Changed prior to

[PATCH] D87774: [flang] Introduce DiagnosticConsumer classes in libflangFrontend

sameeranjoshi added a comment. Thanks for working on it. Few comments inline: 1. For an out-of-tree build, I see `check-flang` target failing with /unittests/Frontend/CompilerInstanceTest.cpp:17:10: fatal error: filesystem: No such file or directory #include ^~~~ I us

[PATCH] D78075: [WIP][Clang][OpenMP] Added support for nowait target in CodeGen

tianshilei1992 updated this revision to Diff 293466. tianshilei1992 added a comment. Fixed the case `target_teams_distribute_simd_codegen.cpp` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78075/new/ https://reviews.llvm.org/D78075 Files: clang/

Re: [PATCH] D87921: Fix -funique-internal-linkage-names to work with -O2 and new pass manager

FWIW I tested check-msan in a -DCMAKE_BUILD_TYPE=Release build on a powerpc64le machine. All tests passed. I cannot connect the failure to the clang patch. On Mon, Sep 21, 2020 at 10:02 PM Sriraman Tallam wrote: > > On Mon, Sep 21, 2020 at 5:58 PM Matt Morehouse via Phabricator > wrote: > > > >

[PATCH] D88100: [analyzer][StdLibraryFunctionsChecker] Separate the signature from the summaries

martong created this revision. martong added reviewers: balazske, Szelethus, steakhal, NoQ, vsavchenko. Herald added subscribers: cfe-commits, ASDenysPetrov, Charusso, gamesh411, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware, xazax.hun, whisperity. Herald ad

[PATCH] D68364: Implement C++20's P0784 (More constexpr containers)

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG0724f8bf47f8: [libc++] Implement C++20's P0784 (More constexpr con

  1   2   >