vit9696 added a comment.
Not sure whether I understood you, do not you already have the common switch by
feature, named spe, in Features["spe"] line for that?
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D49754/new/
https://reviews.llvm.org/D49754
___
Mordante created this revision.
Mordante added a reviewer: rsmith.
Mordante added a project: clang.
Herald added a subscriber: cfe-commits.
Before when the overflow occurred an assertion as triggered. Now check
whether the maximum has been reached and warn properly.
This patch fixes the original
johan-boule added a comment.
Is this going to break symlinks on Windows?
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59415/new/
https://reviews.llvm.org/D59415
___
cfe-commits mailing list
cfe-commits@lists.llvm.or
.org/show_bug.cgi?id=42445
Repository:
rC Clang
https://reviews.llvm.org/D63976
Files:
llvm-9.0.0-20190629/clang/lib/Driver/ToolChains/CommonArgs.cpp
Index: llvm-9.0.0-20190629/clang/lib/Driver/ToolChains/CommonArgs.cpp
==
xbolva00 added a comment.
Good idea, I found this issue a few days ago too.
Please upload the patch with a full context.
Plesse try to add a testcase.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63976/new/
https://reviews.llvm.org/D63976
__
saar.raz updated this revision to Diff 207209.
saar.raz added a comment.
Herald added subscribers: erik.pilkington, mgorny.
Create ASTConstraintSatisfaction for correctly storing constraint satisfaction
data in AST nodes.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.
t; ld.lld: error: -plugin-opt=Oz: number expected, but got 'z'
> clang-9: error: linker command failed with exit code 1 (use -v to see
> invocation)
>
> https://bugs.llvm.org/show_bug.cgi?id=42445
>
>
> Repository:
> rC Clang
>
> https://rev
tejohnson added inline comments.
Comment at: llvm-9.0.0-20190629/clang/lib/Driver/ToolChains/CommonArgs.cpp:395
+ if(OOpt == "s" || OOpt == "z")
+OOpt = "3";
+}
Os/Oz are closer to O2 than O3 (which allows
jhibbits updated this revision to Diff 207214.
jhibbits added a comment.
Made '8548' CPU designation just a stub, to be filled out later. I added it
just for parity with GCC. The 8548 CPU, for GCC, also sets the
__NO_LWSYNC__ macro, which doesn't belong with the SPE change, so will have
to be re