[PATCH] D78454: [clangd] Highlight related control flow.

Gallop, Russell via cfe-commits cfe-commits at lists.llvm.org
Thu May 28 08:17:02 PDT 2020


Hi Sam,

Thanks for looking at this. You could use something like in clang/ ./unittests/Tooling/RecursiveASTVisitorTests/LambdaExpr.cpp:

  if (llvm::Triple(llvm::sys::getDefaultTargetTriple()).isPS4())
    return; // PS4 does not support fastcall.

Regards
Russ


From: Sam McCall <sammccall at google.com>
Sent: 28 May 2020 15:55
To: reviews+D78454+public+f260770db3c2915f at reviews.llvm.org
Cc: Adam Czachorowski <adamcz at google.com>; Gallop, Russell <Russell.Gallop at sony.com>; Ilya Biryukov <iu.biryukov at gmail.com>; Fangrui Song <maskray at google.com>; Jan Korous <jkorous at apple.com>; mgrang.1234 at gmail.com; Alex L <arphaman at gmail.com>; Kadir Çetinkaya <kadircet at google.com>; Utkarsh Saxena <usx at google.com>; cfe-commits <cfe-commits at lists.llvm.org>; Theko Lekena <mlekena at skidmore.edu>; Nicolas Lesser <blitzrakete at gmail.com>; Han Shen <shenhan at google.com>; Haojian Wu <hokein at google.com>
Subject: Re: [PATCH] D78454: [clangd] Highlight related control flow.

On Thu, May 28, 2020 at 4:35 PM Russell Gallop via Phabricator <reviews at reviews.llvm.org<mailto:reviews at reviews.llvm.org>> wrote:
russell.gallop added a comment.

Hi Sam,

It looks like this is causing a failure on the Windows PS4 buildbot: http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast/builds/32606

Please could you take a look? PS4 target disables RTTI, hence exceptions, by default so it is probably related to that.
Thanks!
Hmm, the clangd tests are supposed to be disabled on PS4 for exactly these sorts of reasons. It looks like that's only working for shell tests, not gtests though.
Lit config is a tangled mess... is there an #ifdef we can use to disable just that test on PS4 until I get this sorted out?


Thanks
Russ


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78454/new/

https://reviews.llvm.org/D78454





**********************************************************************
This email and any files transmitted with it are confidential and intended solely for the use of the individual or entity to whom they are addressed. If you have received this email in error please notify siee.postmaster at sony.com<mailto:siee.postmaster at sony.com>
This footnote also confirms that this email message has been checked for all known viruses.
Sony Interactive Entertainment Europe Limited
Registered Office: 10 Great Marlborough Street, London W1F 7LP, United Kingdom
Registered in England: 3277793
**********************************************************************

P Please consider the environment before printing this e-mail
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200528/48bd4652/attachment-0001.html>


More information about the cfe-commits mailing list