[PATCH] D50594: [analyzer] [NFC] Introduce separate targets for testing the analyzer: check-clang-analyzer and check-clang-analyzer-z3

thopre added a comment. Hi George, This commit seems to be causing some testsuite regression on Armv8 (both Aarch64 and ARM) architectures: http://lab.llvm.org:8011/builders/clang-cmake-armv8-full/builds/5450 http://lab.llvm.org:8011/builders/clang-cmake-armv8-quick/builds/5625 Best regards,

[PATCH] D45965: [Targets] Implement getConstraintRegister for ARM and AArch64

thopre added inline comments. Comment at: lib/Basic/Targets/AArch64.h:85-89 + StringRef getConstraintRegister(StringRef Constraint, + StringRef Expression) const override { +return Expression; + } + From what I understood

[PATCH] D45965: [Targets] Implement getConstraintRegister for ARM and AArch64

thopre added a comment. FWIW LGTM. https://reviews.llvm.org/D45965 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D45965: [Targets] Implement getConstraintRegister for ARM and AArch64

thopre added inline comments. Comment at: lib/Basic/Targets/AArch64.h:85-89 + StringRef getConstraintRegister(StringRef Constraint, + StringRef Expression) const override { +return Expression; + } + miyuki wrote: > thopre

[PATCH] D45965: [Targets] Implement getConstraintRegister for ARM and AArch64

thopre accepted this revision. thopre added a comment. This revision is now accepted and ready to land. As I said, LGTM. Please wait a few days (say next Monday) in case anyone else has any objection. https://reviews.llvm.org/D45965 ___

[PATCH] D55428: [Docs] Expand -fstack-protector and -fstack-protector-all info

thopre accepted this revision. thopre added a comment. This revision is now accepted and ready to land. LGTM. Thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55428/new/ https://reviews.llvm.org/D55428 ___ cfe-commits mailing list

[PATCH] D55428: [Docs] Expand -fstack-protector and -fstack-protector-all info

thopre added inline comments. Comment at: include/clang/Driver/Options.td:1636 def fstack_protector_all : Flag<["-"], "fstack-protector-all">, Group, HelpText<"Force the usage of stack protectors for all functions">; def fstack_protector_strong : Flag<["-"],

[PATCH] D55428: [Docs] Expand -fstack-protector and -fstack-protector-all info

thopre added inline comments. Comment at: include/clang/Driver/Options.td:1634-1636 + HelpText<"Uses a stronger heuristic to apply stack protectors to functions " + "that include arrays of any size (and any type), " + "as well as any calls to alloca or the

[PATCH] D61269: [CommandLine] Change help output to prefix long options with `--` instead of `-`. NFC . Part 3 of 5

thopre accepted this revision. thopre added a comment. This revision is now accepted and ready to land. That's much better thank you. I like the stream shift operator overload, nice touch. LGTM but give it a day for other to give feedback. Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D61269: [CommandLine] Change help output to prefix long options with `--` instead of `-`. NFC . Part 3 of 5

thopre added inline comments. Comment at: llvm/lib/Support/CommandLine.cpp:95 + +static size_t argPrefixesSize(size_t len) { + if (len == 1) Any reason why not take a StringRef of the option to compute the prefix of? That would put the .size() gymnastic in one

[PATCH] D68155: [clang][NFC] Make various uses of Regex const

thopre accepted this revision. thopre added a comment. This revision is now accepted and ready to land. LGTM. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68155/new/ https://reviews.llvm.org/D68155 ___ cfe-commits

[PATCH] D71408: [lit] Remove lit's REQUIRES-ANY directive

thopre updated this revision to Diff 233712. thopre added a comment. Herald added subscribers: Sanitizers, cfe-commits, luismarques, apazos, sameer.abuasal, pzheng, s.egerton, lenary, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, delcypher, niosHD,

[PATCH] D71408: [lit] Remove lit's REQUIRES-ANY directive

This revision was automatically updated to reflect the committed changes. Closed by commit rGddd0bb8dba2a: [lit] Remove lits REQUIRES-ANY directive (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71408/new/

[PATCH] D71408: [lit] Remove lit's REQUIRES-ANY directive

thopre added a subscriber: nemanjai. thopre added a comment. Herald added a subscriber: wuzish. In D71408#1787959 , @jdenny wrote: > This breaks the lit test suite. `llvm/utils/lit/tests/shtest-format.py` > needs to be updated for the removed tests.

[PATCH] D68155: [clang][NFC] Make various uses of Regex const

This revision was automatically updated to reflect the committed changes. Closed by commit rGb81cc6032902: [clang][NFC] Make various uses of Regex const (authored by thopre). Changed prior to commit: https://reviews.llvm.org/D68155?vs=26=230079#toc Repository: rG LLVM Github Monorepo

[PATCH] D68155: [clang][NFC] Make various uses of Regex const

thopre added a comment. My sincere apologies, I forgot to change the author in the commit. I don't often commit on behalf of others so didn't think enough. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68155/new/ https://reviews.llvm.org/D68155

[PATCH] D70748: [clang test] Do not assume default target

thopre created this revision. thopre added a reviewer: thegameg. Herald added a project: clang. Herald added a subscriber: cfe-commits. clang test Driver/darwin-opt-record.c assumes the default target is x86_64 by its uses of the -arch x86_64 and -arch x86_64h and thus fail on systems where it is

[PATCH] D70748: [clang test] Do not assume default target

thopre closed this revision. thopre added a comment. Committed as 717140a0dcc651ca2fec23248d1675fb2d388b9c Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70748/new/

[PATCH] D79276: [FileCheck] Support comment directives

thopre accepted this revision. thopre added a comment. LGTM. Sorry for the late review. Comment at: llvm/test/FileCheck/comment/after-words.txt:1 +# Comment prefixes are not recognized at ends of words. + jdenny wrote: > jdenny wrote: > > thopre wrote: > > >

[PATCH] D79276: [FileCheck] Support comment directives

thopre added inline comments. Comment at: llvm/test/FileCheck/comment/after-words.txt:1 +# Comment prefixes are not recognized at ends of words. + How about characters that cannot be part of an identifier? E.g. would "@COM:" be interpreted as a comment? Should

[PATCH] D80933: [clang-format] [PR46157] Wrong spacing of negative literals with use of operator

thopre added a comment. That was a speedy patch! I can confirm that my non reduced testcase (git diff -U0 23ac16cf9bd4cc0bb434efcf6385baf083a2ff7b ^ 23ac16cf9bd4cc0bb434efcf6385baf083a2ff7b

[PATCH] D85367: [clang, test, Darwin] Fix tests expecting Darwin target

This revision was automatically updated to reflect the committed changes. Closed by commit rG6dc863cd858f: [clang, test, Darwin] Fix tests expecting Darwin target (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85367/new/

[PATCH] D85367: [clang, test, Darwin] Fix tests expecting Darwin target

thopre created this revision. thopre added reviewers: zixuw, arphaman, ributzka, steven_wu, dexonsmith. Herald added subscribers: cfe-commits, kristof.beyls. Herald added a project: clang. thopre requested review of this revision. Clang tests Driver/apple-arm64-arch.c and

[PATCH] D81422: Change filecheck default to dump input on failure

thopre accepted this revision. thopre added a comment. In D81422#2080758 , @probinson wrote: > I don't remember the exact reasoning but I believe it had something to do > with bot logs? @jdenny or @thopre might remember. I guess -dump-input just

[PATCH] D92409: [AST][NFC] Silence GCC warning about multiline comments

thopre added a comment. In D92409#2481300 , @rjmccall wrote: > People can't actually just copy/paste from the comment anyway because of the > comment characters on the line, and I don't think anyone will misunderstand > the example if the backslash is

[PATCH] D92409: [AST][NFC] Silence GCC warning about multiline comments

thopre updated this revision to Diff 315080. thopre added a comment. Change approach to remove continuation line instead Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92409/new/ https://reviews.llvm.org/D92409 Files:

[PATCH] D92409: [AST][NFC] Silence GCC warning about multiline comments

This revision was automatically updated to reflect the committed changes. Closed by commit rGacbb3652931a: [AST][NFC] Silence GCC warning about multiline comments (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92409/new/

[PATCH] D92409: [AST][NFC] Silence GCC warning about multiline comments

thopre created this revision. thopre added reviewers: ABataev, rjmccall, rsmith. Herald added a project: clang. Herald added a subscriber: cfe-commits. thopre requested review of this revision. Herald added a reviewer: jdoerfert. Herald added a subscriber: sstefan1. GCC's -Wcomment warning is

[PATCH] D92409: [AST][NFC] Silence GCC warning about multiline comments

thopre added a comment. In D92409#2426160 , @ABataev wrote: > I would just remove this extra symbol. But then the example code is no longer copy/pastable. Note that I expect people to copy/paste but it's nice if it looks authentic. Note that I don't

[PATCH] D92409: [AST][NFC] Silence GCC warning about multiline comments

thopre added a comment. In D92409#2426188 , @rsmith wrote: > Let's just disable this broken GCC warning. IIRC, the corresponding clang > warning covers the actual bugs caused by line continuation in comments, and > doesn't warn if the next line begins

[PATCH] D92384: [AST][NFC] Silence GCC warning about broken strict aliasing rules

This revision was automatically updated to reflect the committed changes. Closed by commit rG507bbc45bba9: [AST][NFC] Silence GCC warning about broken strict aliasing rules (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D92384: [AST][NFC] Silence GCC warning about broken strict aliasing rules

thopre created this revision. thopre added reviewers: jdenny, aaron.ballman, hfinkel, llvm-commits. Herald added a project: clang. Herald added a subscriber: cfe-commits. thopre requested review of this revision. The deserialize() method would trigger the following warning on GCC <7: warning:

[PATCH] D92409: [AST][NFC] Silence GCC warning about multiline comments

thopre added a comment. In D92409#2426196 , @thopre wrote: > In D92409#2426188 , @rsmith wrote: > >> Let's just disable this broken GCC warning. IIRC, the corresponding clang >> warning covers the actual bugs

[PATCH] D92409: [AST][NFC] Silence GCC warning about multiline comments

thopre added a comment. In D92409#2450690 , @rsmith wrote: > In D92409#2450550 , @thopre wrote: > >> In D92409#2426196 , @thopre wrote: >> >>> Is there a way to disable it

[PATCH] D93350: [Test] Fix undef var in catch-undef-behavior.c

thopre created this revision. thopre added reviewers: rsmith, rnk, BillyONeal, spatel. thopre requested review of this revision. Herald added a project: clang. Commit 9e52c43090f8cd980167bbd2719878ae36bcf6b5 removed the directive defining LINE_1600 but left a string substitution to that variable

[PATCH] D93347: [Test] Fix undef var in attr-speculative-load-hardening.c

thopre created this revision. thopre added reviewers: zbrid, chandlerc, EricWF. thopre requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Fix use of undefined variable in CHECK-NOT directive in clang test

[PATCH] D93350: [Test] Fix undef var in catch-undef-behavior.c

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG150fe05db441: [Test] Fix undef var in catch-undef-behavior.c (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D92409: [AST][NFC] Silence GCC warning about multiline comments

thopre added a comment. In D92409#2450897 , @thopre wrote: > In D92409#2450690 , @rsmith wrote: > >> In D92409#2450550 , @thopre wrote: >> >>> In D92409#2426196

[PATCH] D94396: Fix macos target assumption in test

thopre created this revision. thopre added a reviewer: arphaman. thopre requested review of this revision. Herald added a project: clang. Clang test Driver/macos-apple-silicon-slice-link-libs-darwin-only.cpp assumes the target is darwin when the host is darwin which is not necessarily the case,

[PATCH] D93347: [Test] Fix undef var in attr-speculative-load-hardening.c

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

[PATCH] D95948: Stop traping on sNaN in __builtin_isnan

thopre updated this revision to Diff 321107. thopre marked 2 inline comments as done. thopre added a comment. - Fix order of sub operands in comments and use fully parenthesized expression - Explain how LShr is equivalent to < 0 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D94396: Fix macos target assumption in test

thopre added reviewers: hans, jyknight, steven_wu. thopre added a comment. Ping? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94396/new/ https://reviews.llvm.org/D94396 ___ cfe-commits mailing list

[PATCH] D94396: Fix macos target assumption in test

This revision was automatically updated to reflect the committed changes. Closed by commit rG305ac81e1d4b: Fix macos target assumption in test (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94396/new/

[PATCH] D93347: [Test] Fix undef var in attr-speculative-load-hardening.c

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

[PATCH] D95948: Stop traping on sNaN in __builtin_isnan

thopre created this revision. thopre added reviewers: efriedma, kpn, mibintc, sepavloff, rjmccall. Herald added subscribers: dexonsmith, pengfei. thopre requested review of this revision. Herald added projects: clang, LLVM. Herald added a subscriber: llvm-commits. __builtin_isnan currently

[PATCH] D95948: Stop traping on sNaN in __builtin_isnan

thopre marked 2 inline comments as done. thopre added inline comments. Comment at: clang/lib/CodeGen/CGBuiltin.cpp:2988 CodeGenFunction::CGFPOptionsRAII FPOptsRAII(*this, E); -// FIXME: for strictfp/IEEE-754 we need to not trap on SNaN here. Value *V =

[PATCH] D95948: Stop traping on sNaN in __builtin_isnan

thopre marked 2 inline comments as done. thopre added inline comments. Comment at: clang/lib/CodeGen/CGBuiltin.cpp:2995 +Builder.getDefaultConstrainedExcept() == fp::ebIgnore || +!Ty->isIEEE()) { + V = Builder.CreateFCmpUNO(V, V, "cmp"); I'm

[PATCH] D96568: [CFE, SystemZ] Emit s390.tdc instrincic for __builtin_isnan in Constrained FP mode.

thopre added a comment. In D96568#2569296 , @jonpa wrote: >> Sounds good to me. Hopefully I'll get round to __builtin_isinf soon and a >> single hook will make the patch slightly smaller. > > Patch updated to call the new hook testFPKind() and make it

[PATCH] D104854: Introduce intrinsic llvm.isnan

thopre added a comment. Are you planning to do this for the other FP test builtin (isinf, isfinite, isinf_sign, isnormal)? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104854/new/ https://reviews.llvm.org/D104854

[PATCH] D104854: Introduce intrinsic llvm.isnan

thopre added a comment. LGTM (besides comment fix) but I'm not too familiar with the vector side of things Comment at: llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp:6967 + // NaN has all exp bits set and a non zero significand. Therefore: + // isnan(V) == ((exp mask -

[PATCH] D104099: [NewPM] Remove SpeculateAroundPHIs pass from pipeline

thopre added a comment. +1 for this change but being a downstream target only I prefer to let someone else approve this. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104099/new/ https://reviews.llvm.org/D104099

[PATCH] D100226: [funcattrs] Add the maximal set of implied attributes to definitions

thopre added inline comments. Comment at: llvm/test/Transforms/FunctionAttrs/2008-09-03-ReadOnly.ll:14-15 -; CHECK: attributes #0 = { readonly } +; CHECK: attributes #0 = { nofree readonly } +; CHECK: attributes #1 = { readonly } I'm getting #0 for both

[PATCH] D100226: [funcattrs] Add the maximal set of implied attributes to definitions

thopre added inline comments. Comment at: llvm/test/Transforms/FunctionAttrs/2008-09-03-ReadOnly.ll:14-15 -; CHECK: attributes #0 = { readonly } +; CHECK: attributes #0 = { nofree readonly } +; CHECK: attributes #1 = { readonly } jdoerfert wrote: > thopre

[PATCH] D95948: Stop traping on sNaN in __builtin_isnan

thopre updated this revision to Diff 321554. thopre added a comment. Add AArch64 testcase and rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95948/new/ https://reviews.llvm.org/D95948 Files: clang/lib/CodeGen/CGBuiltin.cpp

[PATCH] D95948: Stop traping on sNaN in __builtin_isnan

thopre added inline comments. Comment at: clang/test/CodeGen/aarch64-strictfp-builtins.c:1 +// RUN: %clang_cc1 %s -emit-llvm -ffp-exception-behavior=maytrap -fexperimental-strict-floating-point -o - -triple arm64-none-linux-gnu | FileCheck %s + AArch64 is not

[PATCH] D95948: Stop traping on sNaN in __builtin_isnan

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG00a62547da7e: Stop traping on sNaN in __builtin_isnan (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D99771: [OpenMP, test] Fix use of undef DECL FileCheck var

thopre created this revision. thopre added reviewers: cchen, ABataev, MaskRay. Herald added subscribers: guansong, yaxunl. thopre requested review of this revision. Herald added a reviewer: jdoerfert. Herald added subscribers: cfe-commits, sstefan1. Herald added a project: clang. OpenMP test

[PATCH] D99770: [OpenMP, test] Fix uses of undef S*VAR FileCheck var

thopre updated this revision to Diff 334846. thopre added a comment. Fix one more testcase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99770/new/ https://reviews.llvm.org/D99770 Files: clang/test/OpenMP/for_firstprivate_codegen.cpp

[PATCH] D99770: [OpenMP, test] Fix uses of undef S*VAR FileCheck var

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGd222a07d3023: [OpenMP, test] Fix uses of undef S*VAR FileCheck var (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D99771: [OpenMP, test] Fix use of undef DECL FileCheck var

This revision was automatically updated to reflect the committed changes. Closed by commit rG58e458935ba6: [OpenMP, test] Fix use of undef DECL FileCheck var (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99771/new/

[PATCH] D99775: [OpenMP, test] Fix use of undef VAR_PRIV FileCheck var

thopre created this revision. thopre added reviewers: ABataev, mikerice, jdoerfert, MaskRay. Herald added subscribers: guansong, yaxunl. thopre requested review of this revision. Herald added subscribers: cfe-commits, sstefan1. Herald added a project: clang. Remove the CHECK-NOT directive

[PATCH] D99770: [OpenMP, test] Fix uses of undef S*VAR FileCheck var

thopre created this revision. thopre added reviewers: ABataev, MaskRay, jdoerfert. Herald added subscribers: guansong, yaxunl. thopre requested review of this revision. Herald added subscribers: cfe-commits, sstefan1. Herald added a project: clang. Fix the many cases of use of undefined

[PATCH] D99775: [OpenMP, test] Fix use of undef VAR_PRIV FileCheck var

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG2c3db73341ae: [OpenMP, test] Fix use of undef VAR_PRIV FileCheck var (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D99857: [OpenCL, test] Fix use of undef FileCheck var

This revision was automatically updated to reflect the committed changes. Closed by commit rG828ec9e9e5da: [OpenCL, test] Fix use of undef FileCheck var (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99857/new/

[PATCH] D99898: [clang, test] Fix use of undef FileCheck var

thopre created this revision. thopre added reviewers: ddunbar, MaskRay, rjmccall, spatel, arsenm, hfinkel, void, mcrosier. thopre requested review of this revision. Herald added a subscriber: wdng. Herald added a project: clang. Clang test CodeGen/libcalls.c contains CHECK-NOT directives using a

[PATCH] D99898: [clang, test] Fix use of undef FileCheck var

thopre updated this revision to Diff 335640. thopre added a comment. Rename NUW_RN for CHECK-YES to NUW Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99898/new/ https://reviews.llvm.org/D99898 Files: clang/test/CodeGen/libcalls.c Index:

[PATCH] D99898: [clang, test] Fix use of undef FileCheck var

thopre updated this revision to Diff 335618. thopre added a comment. Change attribute check to a positive match Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99898/new/ https://reviews.llvm.org/D99898 Files: clang/test/CodeGen/libcalls.c

[PATCH] D99898: [clang, test] Fix use of undef FileCheck var

This revision was automatically updated to reflect the committed changes. Closed by commit rGe018698bec36: [clang, test] Fix use of undef FileCheck var (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99898/new/

[PATCH] D99830: [DebugInfo, CallSites, test] Fix use of undef FileCheck var

thopre created this revision. thopre added reviewers: djtodoro, vsk. thopre requested review of this revision. Herald added a project: clang. Clang test CodeGen/debug-info-extern-call.c tries to check for the absence of a sequence of instructions with several CHECK-NOT with one of those

[PATCH] D99838: [C++20, test] Fix use of undef FileCheck variable

thopre created this revision. thopre added reviewers: rsmith, EricWF, akhuang. thopre requested review of this revision. Herald added a project: clang. Commit f495de43bd5da50286da6020e508d106cfc60f57 forgot two lines when

[PATCH] D99839: [C++, test] Fix typo in NSS* vars

thopre created this revision. thopre added reviewers: respindola, pcc, EricWF. thopre requested review of this revision. Herald added a project: clang. The NSS FileCheck variables at the end of the CodeGenCXX/split-stacks.cpp clang testcase are off by 1, resulting in the use of an undefined

[PATCH] D99821: [PGO, test] Fix typo in FileCheck var

This revision was automatically updated to reflect the committed changes. Closed by commit rG95f448aa86cd: [PGO, test] Fix typo in FileCheck var (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99821/new/

[PATCH] D99832: [HIP, test] Fix use of undef FileCheck var

thopre created this revision. thopre added reviewers: yaxunl, ashi1, MaskRay, hliao, pcc. thopre requested review of this revision. Herald added a project: clang. Clang test CodeGenCUDA/kernel-stub-name.cu uses never defined DKERN variable in a CHECK-NOT directive. This commit replace the

[PATCH] D99831: [HIP-Clang, test] Fix use of undef FileCheck var

thopre created this revision. thopre added reviewers: ashi1, yaxunl, MaskRay, hliao, pcc. thopre requested review of this revision. Herald added a project: clang. Commit 8129521318accc44c2a009647572f6ebd3fc56dd changed a line defining PREFIX in clang test CodeGenCUDA/device-stub.cu into a

[PATCH] D99821: [PGO, test] Fix typo in FileCheck var

thopre created this revision. thopre added reviewers: bogner, xur, davidxl. Herald added a subscriber: wenlei. thopre requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D99821

[PATCH] D99857: [OpenCL, test] Fix use of undef FileCheck var

thopre created this revision. thopre added reviewers: hliao, craig.topper, neil.hickey, yaxunl, baldrick. Herald added subscribers: ldrumm, Anastasia. thopre requested review of this revision. Herald added a project: clang. Clang test CodeGenOpenCL/fpmath.cl uses a variable defined in an earlier

[PATCH] D99838: [C++20, test] Fix use of undef FileCheck variable

This revision was automatically updated to reflect the committed changes. Closed by commit rG1cc9d949a123: [C++20, test] Fix use of undef FileCheck variable (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99838/new/

[PATCH] D99832: [HIP, test] Fix use of undef FileCheck var

This revision was automatically updated to reflect the committed changes. Closed by commit rG292726b6443c: [HIP, test] Fix use of undef FileCheck var (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99832/new/

[PATCH] D99831: [HIP-Clang, test] Fix use of undef FileCheck var

This revision was automatically updated to reflect the committed changes. Closed by commit rGa41b5100e438: [HIP-Clang, test] Fix use of undef FileCheck var (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99831/new/

[PATCH] D99830: [DebugInfo, CallSites, test] Fix use of undef FileCheck var

This revision was automatically updated to reflect the committed changes. Closed by commit rG4dd3e0feca92: [DebugInfo, CallSites, test] Fix use of undef FileCheck var (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99830/new/

[PATCH] D97125: Stop traping on sNaN in __builtin_isinf

thopre added a comment. In D97125#2603832 , @thopre wrote: > Requesting review since the logic has changed. This time I've also tested > isfinite against glibc's result. All looks good. Ping? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D98852: [test] Fix variable definition in acle_sve_ld1.sh

This revision was automatically updated to reflect the committed changes. Closed by commit rGe5cd5b352ff4: [test] Fix variable definition in acle_sve_ld1.sh (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98852/new/

[PATCH] D98852: [test] Fix variable definition in acle_sve_ld1.sh

thopre created this revision. thopre added reviewers: andwar, david-arm, sdesmalen, SjoerdMeijer, kmclaughlin. Herald added a subscriber: tschuett. thopre requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Clang test acle_sve_ld1.sh is

[PATCH] D98852: [test] Fix variable definition in acle_sve_ld1.sh

thopre added a comment. In D98852#2634189 , @sdesmalen wrote: > LGTM, thanks for fixing! > > I'm curious, how did you find this case? :) The issue was caught by a patch of mine to FileCheck: https://reviews.llvm.org/D98691 Repository: rG LLVM Github

[PATCH] D97125: Stop traping on sNaN in __builtin_isinf

This revision was automatically updated to reflect the committed changes. Closed by commit rGf60b35340fd7: Stop traping on sNaN in __builtin_isinf (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97125/new/

[PATCH] D93347: [Test] Fix undef var in attr-speculative-load-hardening.c

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG2426b1fa66f9: [Test] Fix undef var in attr-speculative-load-hardening.c (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D97777: Add __builtin_isnan(__fp16) testcase

thopre created this revision. thopre added reviewers: mibintc, kpn, efriedma, sepavloff, rjmccall, jonpa. thopre requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D9 Files:

[PATCH] D97125: Stop traping on sNaN in __builtin_isinf

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1b6eb56aa0ea: Stop traping on sNaN in __builtin_isinf (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D97125: Stop traping on sNaN in __builtin_isinf

thopre added a comment. Ping? Comment at: clang/test/CodeGen/builtin_float_strictfp.c:21 + // NOFP16-NEXT: [[IHALF:%.*]] = load i16, i16* [[LDADDR]], align 2 + // NOFP16-NEXT: [[CONV:%.*]] = call float @llvm.convert.from.fp16.f32(i16 [[IHALF]]) + // NOFP16-NEXT:

[PATCH] D97777: Add __builtin_isnan(__fp16) testcase

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGe77b5c40d576: Add __builtin_isnan(__fp16) testcase (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D97901: [SystemZ] Test for infinity in testFPKind().

thopre added inline comments. Comment at: clang/lib/CodeGen/TargetInfo.cpp:7229 + case Builtin::BI__builtin_isfinite: +Invert = true; +LLVM_FALLTHROUGH; jonpa wrote: > What are these variants all about...? > They were introduced in

[PATCH] D97777: Add __builtin_isnan(__fp16) testcase

thopre updated this revision to Diff 328130. thopre added a comment. Adapt to revert of __builtin_isinf Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D9/new/ https://reviews.llvm.org/D9 Files: clang/test/CodeGen/strictfp_builtins.c

[PATCH] D97777: Add __builtin_isnan(__fp16) testcase

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG52bfe6605ab7: Add __builtin_isnan(__fp16) testcase (authored by thopre). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D97901: [SystemZ] Test for infinity in testFPKind().

thopre added inline comments. Comment at: clang/lib/CodeGen/TargetInfo.cpp:7229 + case Builtin::BI__builtin_isfinite: +Invert = true; +LLVM_FALLTHROUGH; uweigand wrote: > thopre wrote: > > jonpa wrote: > > > What are these variants all

[PATCH] D97125: Stop traping on sNaN in __builtin_isinf

thopre updated this revision to Diff 328170. thopre added a comment. Fix isfinite logic Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97125/new/ https://reviews.llvm.org/D97125 Files: clang/lib/CodeGen/CGBuiltin.cpp

[PATCH] D97125: Stop traping on sNaN in __builtin_isinf

thopre updated this revision to Diff 328171. thopre added a comment. Add isfinite testcases Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97125/new/ https://reviews.llvm.org/D97125 Files: clang/lib/CodeGen/CGBuiltin.cpp

[PATCH] D97901: [SystemZ] Test for infinity in testFPKind().

thopre added inline comments. Comment at: clang/lib/CodeGen/TargetInfo.cpp:7229 + case Builtin::BI__builtin_isfinite: +Invert = true; +LLVM_FALLTHROUGH; thopre wrote: > uweigand wrote: > > thopre wrote: > > > jonpa wrote: > > > > What are

[PATCH] D97125: Stop traping on sNaN in __builtin_isinf

thopre requested review of this revision. thopre added a comment. Requesting review since the logic has changed. This time I've also tested isfinite against glibc's result. All looks good. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97125/new/

[PATCH] D97125: Stop traping on sNaN in __builtin_isinf

thopre added inline comments. Comment at: clang/test/CodeGen/builtin_float_strictfp.c:21 + // NOFP16-NEXT: [[IHALF:%.*]] = load i16, i16* [[LDADDR]], align 2 + // NOFP16-NEXT: [[CONV:%.*]] = call float @llvm.convert.from.fp16.f32(i16 [[IHALF]]) + // NOFP16-NEXT:

[PATCH] D97125: Stop traping on sNaN in __builtin_isinf

thopre updated this revision to Diff 325718. thopre added a comment. - Add half testcase - Fixup existing testcase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97125/new/ https://reviews.llvm.org/D97125 Files: clang/lib/CodeGen/CGBuiltin.cpp

  1   2   >