[PATCH] D71141: [Wdocumentation] Use C++14 deprecated attribute

Mordante marked an inline comment as done. Mordante added inline comments. Comment at: clang/lib/AST/CommentSema.cpp:693 +StringRef AttributeSpelling = +CPlusPlus14 ? "[[deprecated]]" : "__attribute__((deprecated))"; if (PP) { aaron.ballman wrote

[PATCH] D70553: [clang-apply-replacements] Add command line option to overwrite readonly files.

aaron.ballman added a comment. In D70553#1773046 , @zturner wrote: > In D70553#1757862 , @aaron.ballman > wrote: > > > Can you add a test case for this functionality? > > > The reason there's no test case is becaus

[PATCH] D71080: [NFC] Separate getLastArgIntValue to Basic

yaxunl marked 5 inline comments as done. yaxunl added inline comments. Comment at: clang/include/clang/Basic/OptionUtils.h:24 + +class ArgList; + tra wrote: > What are the rules on using LLVM headers here? Can we just include > llvm/Option/ArgList.h instead? In

[PATCH] D70877: [WebAssebmly][MC] Support .import_name/.import_field asm directives

sbc100 added a comment. Ping .. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70877/new/ https://reviews.llvm.org/D70877 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/c

[PATCH] D71091: Make sure that the implicit arguments for direct methods have been setup

MadCoder updated this revision to Diff 232638. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71091/new/ https://reviews.llvm.org/D71091 Files: clang/lib/CodeGen/CGDebugInfo.cpp clang/lib/CodeGen/CGObjCMac.cpp clang/lib/Sema/SemaDeclObjC.cpp clang/lib/Serialization/ASTWriterDecl.c

[clang] 040c39d - [analyzer] Fix false positive on introspection of a block's internal layout.

Author: Artem Dergachev Date: 2019-12-06T13:24:20-08:00 New Revision: 040c39d50fb9c60de9020caf86e1a1fccfd6f861 URL: https://github.com/llvm/llvm-project/commit/040c39d50fb9c60de9020caf86e1a1fccfd6f861 DIFF: https://github.com/llvm/llvm-project/commit/040c39d50fb9c60de9020caf86e1a1fccfd6f861.dif

[PATCH] D71080: [NFC] Separate getLastArgIntValue to Basic

yaxunl updated this revision to Diff 232639. yaxunl marked 2 inline comments as done. yaxunl added a comment. revised by Artem's comments CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71080/new/ https://reviews.llvm.org/D71080 Files: clang/include/clang/Basic/OptionUtils.h clang/in

[PATCH] D71091: Make sure that the implicit arguments for direct methods have been setup

MadCoder added inline comments. Comment at: clang/lib/Sema/SemaDeclObjC.cpp:4839 + ObjCMethod->createImplicitParams(Context, ObjCMethod->getClassInterface()); + MadCoder wrote: > liuliu wrote: > > Yeah, I applied this change. As I said, for my trivial example

[PATCH] D71091: Make sure that the implicit arguments for direct methods have been setup

MadCoder updated this revision to Diff 232641. MadCoder marked an inline comment as done. MadCoder added a comment. woops a bogus hunk went in CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71091/new/ https://reviews.llvm.org/D71091 Files: clang/lib/CodeGen/CGDebugInfo.cpp clang/lib

[PATCH] D70804: [Frontend] Allow OpenMP offloading to aarch64

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

[PATCH] D70469: [attributes] [analyzer] Add handle related attributes

aaron.ballman added a comment. In D70469#1766084 , @xazax.hun wrote: > @aaron.ballman > > Type attributes are definitely more flexible as in they can be applied in > more contexts. These attributes, however, make no sense outside of a function > declarat

[PATCH] D71142: [Sema] Validate large bitfields

aaron.ballman added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:5186-5189 +def warn_bitfield_width_exceeds_maximum_width: Error< + "width of bit-field %0 doesn't fit in a 64 bit unsigned integer">; +def warn_anon_bitfield_width_exceeds_maximum_w

[PATCH] D71091: Make sure that the implicit arguments for direct methods have been setup

arphaman added inline comments. Comment at: clang/lib/Serialization/ASTWriterDecl.cpp:674 } Record.push_back(D->isInstanceMethod()); Record.push_back(D->isVariadic()); @MadCoder Given the fact that you're now setting SelfDecl and CmdDecl for declaration

[PATCH] D71091: Make sure that the implicit arguments for direct methods have been setup

MadCoder updated this revision to Diff 232644. MadCoder marked an inline comment as done. MadCoder added a comment. @liuliu I wouldn't be surprised this addresses your issues if snapchat is using PCH CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71091/new/ https://reviews.llvm.org/D710

[PATCH] D71091: Make sure that the implicit arguments for direct methods have been setup

MadCoder added inline comments. Comment at: clang/lib/Serialization/ASTWriterDecl.cpp:674 } Record.push_back(D->isInstanceMethod()); Record.push_back(D->isVariadic()); arphaman wrote: > @MadCoder Given the fact that you're now setting SelfDecl and CmdDec

[PATCH] D71091: Make sure that the implicit arguments for direct methods have been setup

arphaman added a comment. In D71091#1773494 , @MadCoder wrote: > @liuliu I wouldn't be surprised this addresses your issues if snapchat is > using PCH This also impacts modules, so builds that use could've been affected. CHANGES SINCE LAST ACTION ht

[PATCH] D71091: Make sure that the implicit arguments for direct methods have been setup

arphaman accepted this revision. arphaman added a comment. This revision is now accepted and ready to land. LGTM, thanks CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71091/new/ https://reviews.llvm.org/D71091 ___ cfe-commits mailing list c

[clang] f3efd69 - [ObjC] Make sure that the implicit arguments for direct methods have been setup

Author: Alex Lorenz Date: 2019-12-06T14:28:28-08:00 New Revision: f3efd6957474bfd3b9b232ac6e4b3608174c3b79 URL: https://github.com/llvm/llvm-project/commit/f3efd6957474bfd3b9b232ac6e4b3608174c3b79 DIFF: https://github.com/llvm/llvm-project/commit/f3efd6957474bfd3b9b232ac6e4b3608174c3b79.diff L

[PATCH] D71091: Make sure that the implicit arguments for direct methods have been setup

This revision was automatically updated to reflect the committed changes. Closed by commit rGf3efd6957474: [ObjC] Make sure that the implicit arguments for direct methods have been setup (authored by arphaman). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llv

[clang] 0a717d5 - Make it possible control matcher traversal kind with ASTContext

Author: Stephen Kelly Date: 2019-12-06T23:11:32Z New Revision: 0a717d5b5d31fc2d5bc98ca695031fb09e65beb0 URL: https://github.com/llvm/llvm-project/commit/0a717d5b5d31fc2d5bc98ca695031fb09e65beb0 DIFF: https://github.com/llvm/llvm-project/commit/0a717d5b5d31fc2d5bc98ca695031fb09e65beb0.diff LOG:

[PATCH] D70877: [WebAssebmly][MC] Support .import_name/.import_field asm directives

This revision was automatically updated to reflect the committed changes. Closed by commit rGb4f4e370b59a: [WebAssebmly][MC] Support .import_name/.import_field asm directives (authored by sbc100). Changed prior to commit: https://reviews.llvm.org/D70877?vs=231619&id=232660#toc Repository: rG

[PATCH] D61837: Make it possible control matcher traversal kind with ASTContext

This revision was automatically updated to reflect the committed changes. Closed by commit rG0a717d5b5d31: Make it possible control matcher traversal kind with ASTContext (authored by stephenkelly). Changed prior to commit: https://reviews.llvm.org/D61837?vs=230678&id=232661#toc Repository:

[PATCH] D61837: Make it possible control matcher traversal kind with ASTContext

steveire added a comment. I tried using DenseMap, but couldn't make it compile. I stuck with std::map for now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61837/new/ https://reviews.llvm.org/D61837

[clang-tools-extra] 60573ae - Remove Expr.h include from ASTContext.h, NFC

Author: Reid Kleckner Date: 2019-12-06T15:30:49-08:00 New Revision: 60573ae6fe509b618dc6a2c5c55d921bccd77608 URL: https://github.com/llvm/llvm-project/commit/60573ae6fe509b618dc6a2c5c55d921bccd77608 DIFF: https://github.com/llvm/llvm-project/commit/60573ae6fe509b618dc6a2c5c55d921bccd77608.diff

[PATCH] D71152: [analyzer] Keep track of escaped locals.

xazax.hun added a comment. So some of the questions that came to my mind: 1. Should this be exposed to the checkers? 2. Where should we actually have this trait registered? ProgramState? ExprEngine? Both of them need access, and due to layering I feel like it cannot be in ExprEngine but I might

[PATCH] D71152: [analyzer] Keep track of escaped locals.

xazax.hun created this revision. xazax.hun added reviewers: NoQ, Szelethus, baloghadamsoftware, dcoughlin, haowei. xazax.hun added a project: clang. Herald added subscribers: Charusso, gamesh411, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, rnkovacs, szepet. xazax.hun added a comment. So some

[clang] e1578fd - [Sema][X86] Consider target attribute into the checks in validateOutputSize and validateInputSize.

Author: Craig Topper Date: 2019-12-06T15:30:59-08:00 New Revision: e1578fd2b79fe5af5f80c0c166a8abd0f816c022 URL: https://github.com/llvm/llvm-project/commit/e1578fd2b79fe5af5f80c0c166a8abd0f816c022 DIFF: https://github.com/llvm/llvm-project/commit/e1578fd2b79fe5af5f80c0c166a8abd0f816c022.diff

[PATCH] D68627: [Sema][X86] Consider target attribute into the checks in validateOutputSize and validateInputSize.

This revision was automatically updated to reflect the committed changes. Closed by commit rGe1578fd2b79f: [Sema][X86] Consider target attribute into the checks in validateOutputSize and… (authored by craig.topper). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://review

[PATCH] D71082: Allow system header to provide their own implementation of some builtin

serge-sans-paille updated this revision to Diff 232667. serge-sans-paille added a comment. Fix interaction with fortify warnings (cc @efriedma) Also I've checked interactions with the test suite I wrote here https://github.com/serge-sans-paille/fortify-test-suite/ and clang now passes all the d

[PATCH] D70973: [OPENMP50]Treat context selectors as expressions, not just strings.

jdoerfert added inline comments. Comment at: clang/include/clang/Basic/DiagnosticParseKinds.td:1230 - "unknown '%0' device kind trait in the 'device' context selector set, expected" - " one of 'host', 'nohost', 'cpu', 'gpu' or 'fpga'">; ABataev wrote: > jdoe

[clang-tools-extra] b22d8ae - Use ASTDumper to dump the AST from clang-query

Author: Stephen Kelly Date: 2019-12-06T23:38:56Z New Revision: b22d8ae7f436bfe63b28ceddea743071a6601eb1 URL: https://github.com/llvm/llvm-project/commit/b22d8ae7f436bfe63b28ceddea743071a6601eb1 DIFF: https://github.com/llvm/llvm-project/commit/b22d8ae7f436bfe63b28ceddea743071a6601eb1.diff LOG:

[clang] eff08f4 - Revert "[Sema][X86] Consider target attribute into the checks in validateOutputSize and validateInputSize."

Author: Reid Kleckner Date: 2019-12-06T15:42:14-08:00 New Revision: eff08f40976e177923fe95759917e59375458f71 URL: https://github.com/llvm/llvm-project/commit/eff08f40976e177923fe95759917e59375458f71 DIFF: https://github.com/llvm/llvm-project/commit/eff08f40976e177923fe95759917e59375458f71.diff

[PATCH] D68627: [Sema][X86] Consider target attribute into the checks in validateOutputSize and validateInputSize.

rnk added inline comments. Comment at: clang/include/clang/AST/ASTContext.h:2826 + /// valid feature names. + TargetAttr::ParsedTargetAttr + filterFunctionTargetAttrs(const TargetAttr *TD) const; I reverted this because it requires TargetAttr to be complete he

[PATCH] D62056: Use ASTDumper to dump the AST from clang-query

steveire closed this revision. steveire added a comment. Committed in b22d8ae7f436bfe63 . I don't know why this review didn't automatically update. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://rev

[PATCH] D71152: [analyzer] Keep track of escaped locals.

NoQ added a comment. I've been hiding these internal traits in ExprEngine lately but i don't have a strong preference. I don't see any immediate need to expose this info to the checkers, nor do i see a need to actively hide it. And, yeah, needs dead region cleanup :) Comment

[PATCH] D71152: [analyzer] Keep track of escaped locals.

NoQ added a comment. Hiding the trait in `ExprEngine` has the additional benefit of disallowing *direct* access to the trait - it can only be accessed through getters. It also doesn't prevent us from exposing the trait to checkers as a method on `ProgramState` in the future, because `ProgramSta

[PATCH] D71152: [analyzer] Keep track of escaped locals.

NoQ added inline comments. Comment at: clang/lib/StaticAnalyzer/Core/ProgramState.cpp:48-61 +namespace { +struct EscapedLocals{}; +} // namespace + +template <> +struct ProgramStateTrait : + public ProgramStatePartialTrait> { Wait, you are preventing direct acce

[PATCH] D70973: [OPENMP50]Treat context selectors as expressions, not just strings.

ABataev marked an inline comment as done. ABataev added inline comments. Comment at: clang/include/clang/Basic/DiagnosticParseKinds.td:1230 - "unknown '%0' device kind trait in the 'device' context selector set, expected" - " one of 'host', 'nohost', 'cpu', 'gpu' or 'fpga'">;

[PATCH] D71142: [Sema] Validate large bitfields

rsmith added a comment. It seems fine and reasonable to reject this as a violation of an implementation limit. Can we actually support the full range 2^64 bits range, or would it be wiser to pick a smaller limit? (There's at least no point in allowing bit-widths that would mean the size of the

[clang] 2e8dc85 - Add matchDynamic convenience functions

Author: Stephen Kelly Date: 2019-12-07T00:01:29Z New Revision: 2e8dc8590d8b412a131b127e7aa4aad0d7fc7fa0 URL: https://github.com/llvm/llvm-project/commit/2e8dc8590d8b412a131b127e7aa4aad0d7fc7fa0 DIFF: https://github.com/llvm/llvm-project/commit/2e8dc8590d8b412a131b127e7aa4aad0d7fc7fa0.diff LOG:

[PATCH] D71154: Driver: Don't look for libc++ headers in the install directory on Android.

pcc created this revision. pcc added reviewers: danalbert, srhines. Herald added a reviewer: EricWF. Herald added a subscriber: ldionne. Herald added a project: clang. The NDK uses a separate set of libc++ headers in the sysroot. Any headers in the installation directory are not going to work on A

[PATCH] D54406: Add matchDynamic convenience functions

steveire closed this revision. steveire added a comment. Herald added a project: clang. Committed in 2e8dc8590d8b412 Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54406/new/ https://reviews.ll

[PATCH] D70411: [analyzer] CERT: StrChecker: 31.c

Charusso updated this revision to Diff 232671. Charusso marked 10 inline comments as done. Charusso added a comment. - Make the checker alpha. - Add docs. - Copy-paste @NoQ's test. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70411/new/ https://reviews.llvm.org/D70411 Files: clang/d

[PATCH] D70411: [analyzer] CERT: StrChecker: 31.c

Charusso added a comment. Please let us measure your examples at first to have a consensus what we would like to see from this checker. In D70411#1769803 , @NoQ wrote: > In D70411#1769628 , @Charusso wrote: > > >

[PATCH] D71033: [analyzer] CERT: StrChecker: 32.c

Charusso updated this revision to Diff 232672. Charusso edited the summary of this revision. Charusso added a comment. - Add docs. - Move to alpha. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71033/new/ https://reviews.llvm.org/D71033 Files: clang/docs/analyzer/checkers.rst clang

[PATCH] D71152: [analyzer] Keep track of escaped locals.

xazax.hun updated this revision to Diff 232675. xazax.hun added a comment. - Add cleanup. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71152/new/ https://reviews.llvm.org/D71152 Files: clang/include/clang/StaticAnalyzer/Core/PathSensitive/ProgramState.h clang/lib/StaticAnalyzer/Co

[PATCH] D70520: [WebAssembly] Add new `export_name` clang attribute for controlling wasm export names

sbc100 updated this revision to Diff 232673. sbc100 added a comment. Rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70520/new/ https://reviews.llvm.org/D70520 Files: clang/include/clang/Basic/Attr.td clang/include/clang/Basic/AttrDocs.td

[PATCH] D71155: [analyzer] CERT: StrChecker: 30.c

Charusso created this revision. Charusso added a reviewer: NoQ. Charusso added a project: clang. Herald added subscribers: cfe-commits, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun. Charusso added a comment. Charusso added a parent revision: D710

[PATCH] D71155: [analyzer] CERT: StrChecker: 30.c

Charusso added a comment. Examples generated by CodeChecker from the `curl` project: F10986729: str30-c.tar.gz Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71155/new/ https://reviews.llvm.org/D71155 ___

[clang] 5253d91 - [c++20] Determine whether a defaulted comparison should be deleted or

Author: Richard Smith Date: 2019-12-06T16:32:48-08:00 New Revision: 5253d9138eb31252594f5e14133df731551839c7 URL: https://github.com/llvm/llvm-project/commit/5253d9138eb31252594f5e14133df731551839c7 DIFF: https://github.com/llvm/llvm-project/commit/5253d9138eb31252594f5e14133df731551839c7.diff

[PATCH] D71154: Driver: Don't look for libc++ headers in the install directory on Android.

merge_guards_bot added a comment. Build result: pass - 60573 tests passed, 0 failed and 726 were skipped. Log files: console-log.txt , CMakeCache.txt

[PATCH] D71152: [analyzer] Keep track of escaped locals.

xazax.hun marked an inline comment as done. xazax.hun added inline comments. Comment at: clang/lib/StaticAnalyzer/Core/ProgramState.cpp:48-61 +namespace { +struct EscapedLocals{}; +} // namespace + +template <> +struct ProgramStateTrait : + public ProgramStatePartialTrait> { ---

[PATCH] D71154: Driver: Don't look for libc++ headers in the install directory on Android.

danalbert requested changes to this revision. danalbert added inline comments. This revision now requires changes to proceed. Comment at: clang/test/Driver/android-no-installed-libcxx.cpp:8 +// RUN: -stdlib=libc++ -fsyntax-only %s -### 2>&1 | FileCheck %s +// CHECK-NOT: "-inter

[PATCH] D71154: Driver: Don't look for libc++ headers in the install directory on Android.

pcc marked an inline comment as done. pcc added inline comments. Comment at: clang/test/Driver/android-no-installed-libcxx.cpp:8 +// RUN: -stdlib=libc++ -fsyntax-only %s -### 2>&1 | FileCheck %s +// CHECK-NOT: "-internal-isystem" "{{.*}}v1" danalbert wrote: > T

[PATCH] D70973: [OPENMP50]Treat context selectors as expressions, not just strings.

jdoerfert added inline comments. Comment at: clang/include/clang/Basic/DiagnosticParseKinds.td:1230 - "unknown '%0' device kind trait in the 'device' context selector set, expected" - " one of 'host', 'nohost', 'cpu', 'gpu' or 'fpga'">; ABataev wrote: > jdoe

[PATCH] D71154: Driver: Don't look for libc++ headers in the install directory on Android.

danalbert accepted this revision. danalbert added inline comments. This revision is now accepted and ready to land. Comment at: clang/test/Driver/android-no-installed-libcxx.cpp:8 +// RUN: -stdlib=libc++ -fsyntax-only %s -### 2>&1 | FileCheck %s +// CHECK-NOT: "-internal-isyste

[PATCH] D71159: [Attr] Move ParsedTargetAttr out of the TargetAttr class

craig.topper created this revision. craig.topper added reviewers: rnk, echristo, erichkeane. Need to forward declare it in ASTContext.h for D68627 , so it can't be a nested struct. https://reviews.llvm.org/D71159 Files: clang/include/clang/AST/Attr.h clang/

[PATCH] D70520: [WebAssembly] Add new `export_name` clang attribute for controlling wasm export names

sbc100 added a comment. Ping .. I think this is good to go now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70520/new/ https://reviews.llvm.org/D70520 ___ cfe-commits mailing list cfe-commits@lists.l

[PATCH] D70973: [OPENMP50]Treat context selectors as expressions, not just strings.

ABataev marked an inline comment as done. ABataev added inline comments. Comment at: clang/include/clang/Basic/DiagnosticParseKinds.td:1230 - "unknown '%0' device kind trait in the 'device' context selector set, expected" - " one of 'host', 'nohost', 'cpu', 'gpu' or 'fpga'">;

[PATCH] D70839: [clang][IFS] Claiming -emit-merged-ifs in clang driver when -c is used.

plotfi abandoned this revision. plotfi added a comment. This commit is no good. I got some wrong inspiration from a lot of the code claiming random args in the driver. abandoning. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70839/new/ https://re

[PATCH] D70469: [attributes] [analyzer] Add handle related attributes

xazax.hun updated this revision to Diff 232681. xazax.hun marked 7 inline comments as done. xazax.hun added a comment. - Make the annotation acceptable on both types and declarations. - Fix some TODOs. - Teach TypePrinter to print the annotations. - Address review comments. CHANGES SINCE LAST AC

[PATCH] D70469: [attributes] [analyzer] Add handle related attributes

xazax.hun added inline comments. Comment at: clang/lib/Sema/SemaType.cpp:7412 + diag::warn_type_attribute_wrong_type_str) +<< Attr.getAttrName()->getName() << "non-function" << CurType; +return; aaron.ballman wrote: > You shoul

[PATCH] D71159: [Attr] Move ParsedTargetAttr out of the TargetAttr class

erichkeane added a comment. Should this be in a namespace? Can't tell from the header file, is it at least in clang? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71159/new/ https://reviews.llvm.org/D71159 ___ cfe-commits mailing list cfe-

[clang] 198fbcb - Driver: Don't look for libc++ headers in the install directory on Android.

Author: Peter Collingbourne Date: 2019-12-06T18:24:23-08:00 New Revision: 198fbcb817492ff45946e3f7517de15e8cdf0607 URL: https://github.com/llvm/llvm-project/commit/198fbcb817492ff45946e3f7517de15e8cdf0607 DIFF: https://github.com/llvm/llvm-project/commit/198fbcb817492ff45946e3f7517de15e8cdf0607

[PATCH] D71154: Driver: Don't look for libc++ headers in the install directory on Android.

This revision was automatically updated to reflect the committed changes. Closed by commit rG198fbcb81749: Driver: Don't look for libc++ headers in the install directory on Android. (authored by pcc). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D711

[PATCH] D71159: [Attr] Move ParsedTargetAttr out of the TargetAttr class

craig.topper marked an inline comment as done. craig.topper added inline comments. Comment at: clang/include/clang/AST/Attr.h:359 } } // end namespace clang There's an ending namespace comment here and there's no namespace starting since the end of my chan

[PATCH] D62731: Add support for options -frounding-math, -ftrapping-math, -ffp-model=, and -ffp-exception-behavior=, : Specify floating point behavior

michele.scandale added a comment. I've noticed you removed the change for `CompilerInvocation.cpp` about the initialization of the codegen option `NoTrappingMath`. Was that an accident? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62731/new/ http

[PATCH] D70157: Align branches within 32-Byte boundary

LuoYuanke added a comment. In D70157#1773180 , @reames wrote: > Recording something so I don't forget it when we get back to the prefix > padding version. The write up on the bundle align mode stuff mentions a > concerning memory overhead for the featur

<    1   2