MipsFastISel::fastLowerArguments looks very broken and is likely the
culprit here:
const ArrayRef GPR32ArgRegs = {Mips::A0, Mips::A1, Mips::A2,
Mips::A3};
const ArrayRef FGR32ArgRegs = {Mips::F12, Mips::F14};
const ArrayRef AFGR64ArgRegs = {Mips::D
Thanks Richard,
I was wondering how a Clang change can break the Backend tests given that
the tests are all pre-checked IR files.
Failing Tests (7):
LLVM :: CodeGen/Mips/Fast-ISel/fastalloca.ll
LLVM :: CodeGen/Mips/Fast-ISel/fastcc-miss.ll
LLVM :: CodeGen/Mips/Fast-ISel/memtest1.ll
Author: rsmith
Date: Thu Apr 5 14:49:20 2018
New Revision: 329352
URL: http://llvm.org/viewvc/llvm-project?rev=329352&view=rev
Log:
Add a couple more tests for DR372.
Modified:
cfe/trunk/test/CXX/drs/dr3xx.cpp
Modified: cfe/trunk/test/CXX/drs/dr3xx.cpp
URL:
http://llvm.org/viewvc/llvm-proj
EricWF added inline comments.
Comment at: lib/AST/ItaniumMangle.cpp:3250-3251
- mangleType(T->getBaseType());
+ for (auto Ty : T->getArgs())
+mangleType(Ty);
}
EricWF wrote:
> rsmith wrote:
> > We need manglings to be self-delimiting, and we can't tell w
Author: eugenezelenko
Date: Thu Apr 5 15:15:42 2018
New Revision: 329357
URL: http://llvm.org/viewvc/llvm-project?rev=329357&view=rev
Log:
[Sema] Revert r329346 because of memory sanitizer failures.
Modified:
cfe/trunk/include/clang/Sema/DeclSpec.h
cfe/trunk/include/clang/Sema/SemaFixItU
Author: dyung
Date: Thu Apr 5 15:58:14 2018
New Revision: 329361
URL: http://llvm.org/viewvc/llvm-project?rev=329361&view=rev
Log:
Fix test added in r329301 to work properly with Windows paths.
Modified:
cfe/trunk/test/ASTMerge/struct/test.c
Modified: cfe/trunk/test/ASTMerge/struct/test.c
U
efriedma updated this revision to Diff 141237.
efriedma added a comment.
Compute arch used passed-in CPU, not the global arch.
Repository:
rC Clang
https://reviews.llvm.org/D45240
Files:
lib/Basic/Targets/ARM.cpp
test/CodeGen/arm-long-calls.c
test/CodeGen/arm-no-movt.c
test/CodeGen/a
arphaman added a comment.
In https://reviews.llvm.org/D40983#1059087, @arphaman wrote:
> In https://reviews.llvm.org/D40983#968796, @bruno wrote:
>
> > Makes sense, LGTM.
> >
> > Should we add documentation explaining how to use this? I'm fine if it
> > comes in a follow up commit.
>
>
> Sorry,
arphaman added a comment.
In https://reviews.llvm.org/D40983#968796, @bruno wrote:
> Makes sense, LGTM.
>
> Should we add documentation explaining how to use this? I'm fine if it comes
> in a follow up commit.
Sorry, just got time to get back to this now. Thanks for the review! I'll add a
doc
saar.raz updated this revision to Diff 141248.
saar.raz added a comment.
Adjusted to piecewise substitution.
- Constraint satisfaction will no longer happen for depenent CSEs (was
originally needed for normalization, but not worth the trouble with the new
piecewise substitution and the fact tha
This revision was automatically updated to reflect the committed changes.
Closed by commit rC329366: CMake option to allow enabling experimental new pass
manager by default (authored by phosek, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D44330?vs=137844&id=141249#toc
Rep
Author: phosek
Date: Thu Apr 5 17:53:00 2018
New Revision: 329366
URL: http://llvm.org/viewvc/llvm-project?rev=329366&view=rev
Log:
CMake option to allow enabling experimental new pass manager by default
This CMake flag allows setting the default value for the
-f[no]-experimental-new-pass-manage
manojgupta added a comment.
I think the tests are already broken in Gentoo when clang is installed in
/usr/bin even without this patch. The tests only work if clang binary is not
installed in /usr/bin.
RootCause is the existing lines in Gnu.cpp:
// Then look for gcc installed alongs
efriedma updated this revision to Diff 141255.
efriedma added a comment.
Get rid of the test changes.
They were broken in multiple ways, and weren't really useful anyway because the
targets where you would want to use this can't run the libcxx testsuite anyway
(because they don't have an operat
weimingz added a comment.
Thanks Eli!
Repository:
rCXX libc++
https://reviews.llvm.org/D41316
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
dberris updated this revision to Diff 141265.
dberris added a comment.
- fixup: Rebase, and re-do OpenBSD specific changes
https://reviews.llvm.org/D45243
Files:
clang/include/clang/Driver/XRayArgs.h
clang/lib/Driver/ToolChains/CommonArgs.cpp
clang/lib/Driver/ToolChains/CommonArgs.h
cla
Author: dberris
Date: Thu Apr 5 20:53:04 2018
New Revision: 329372
URL: http://llvm.org/viewvc/llvm-project?rev=329372&view=rev
Log:
[XRay][clang] Consolidate runtime and link-time flag processing (NFC)
Summary:
This change fixes http://llvm.org/PR36985 to define a single place in
CommonArgs.{h,
This revision was automatically updated to reflect the committed changes.
Closed by commit rL329372: [XRay][clang] Consolidate runtime and link-time flag
processing (NFC) (authored by dberris, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm
dberris created this revision.
dberris added reviewers: eizan, echristo, chandlerc.
This change introduces `-fxray-link-deps` and `-fnoxray-link-deps`. The
`-fnoxray-link-deps` allows for directly controlling which specific XRay
runtime to link. The default is for clang to link the XRay runtime th
Author: marshall
Date: Thu Apr 5 21:43:27 2018
New Revision: 329375
URL: http://llvm.org/viewvc/llvm-project?rev=329375&view=rev
Log:
Mark as "In progress"
Modified:
libcxx/trunk/www/cxx2a_status.html
Modified: libcxx/trunk/www/cxx2a_status.html
URL:
http://llvm.org/viewvc/llvm-project/l
Author: dberris
Date: Thu Apr 5 22:28:54 2018
New Revision: 329376
URL: http://llvm.org/viewvc/llvm-project?rev=329376&view=rev
Log:
[XRay][clang] Add a flag to enable/disable linking XRay deps explicitly
Summary:
This change introduces `-fxray-link-deps` and `-fnoxray-link-deps`. The
`-fnoxray-
This revision was automatically updated to reflect the committed changes.
Closed by commit rL329376: [XRay][clang] Add a flag to enable/disable linking
XRay deps explicitly (authored by dberris, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.ll
Author: dberris
Date: Thu Apr 5 23:09:57 2018
New Revision: 329378
URL: http://llvm.org/viewvc/llvm-project?rev=329378&view=rev
Log:
[XRay][clang] Only run driver test for Linux and FreeBSD
This is a follow-up to D45354, which we should have only been running on
Linux and FreeBSD for specific ta
djasper added a comment.
Please read:
https://clang.llvm.org/docs/ClangFormatStyleOptions.html#adding-additional-style-options
In this case in particular, I would be very interested in a style guide that
defines how Allman brace style and lambdas work together. IMO, it has so many
corner ca
101 - 124 of 124 matches
Mail list logo
