Hi Galina,
I'm sad that they appear to have been abandoned but no objections from me as
I'm not the admin for any of these anymore. A couple of them were taken over by
a colleague when I left but he too has left MIPS since then. Simon Atanasyan
might be able to find the owners if there still ar
Author: Daniel Sanders
Date: 2020-08-03T13:18:52-07:00
New Revision: 7209f83112db4dbe15d8328705f9d2aff0624fbd
URL:
https://github.com/llvm/llvm-project/commit/7209f83112db4dbe15d8328705f9d2aff0624fbd
DIFF:
https://github.com/llvm/llvm-project/commit/7209f83112db4dbe15d8328705f9d2aff0624fbd.diff
Author: Daniel Sanders
Date: 2020-08-03T16:55:00-07:00
New Revision: 1beb00db1f5197efb73f839da681b8e439f37628
URL:
https://github.com/llvm/llvm-project/commit/1beb00db1f5197efb73f839da681b8e439f37628
DIFF:
https://github.com/llvm/llvm-project/commit/1beb00db1f5197efb73f839da681b8e439f37628.diff
Author: dsanders
Date: Fri Aug 30 13:01:59 2019
New Revision: 370512
URL: http://llvm.org/viewvc/llvm-project?rev=370512&view=rev
Log:
[clang-tidy] Add llvm-prefer-register-over-unsigned to clang-tidy
Summary:
This clang-tidy check is looking for unsigned integer variables whose
initializer
star
Author: dsanders
Date: Fri Aug 30 13:47:02 2019
New Revision: 370527
URL: http://llvm.org/viewvc/llvm-project?rev=370527&view=rev
Log:
Make add_new_check.py's insertion of registerCheck<> match the sort order
Summary:
Following on from review comments in D65919 about the ordering
of the registerC
Author: Daniel Sanders
Date: 2019-12-03T15:44:01-08:00
New Revision: 327894859cc41c1730807f8a179aa880203262f5
URL:
https://github.com/llvm/llvm-project/commit/327894859cc41c1730807f8a179aa880203262f5
DIFF:
https://github.com/llvm/llvm-project/commit/327894859cc41c1730807f8a179aa880203262f5.diff
Author: dsanders
Date: Tue Sep 15 08:17:40 2015
New Revision: 247683
URL: http://llvm.org/viewvc/llvm-project?rev=247683&view=rev
Log:
Replace Triple with a new TargetTuple in MCTargetDesc/* and related. NFC.
Summary:
This is the first patch in the series to migrate Triple's (which are ambiguous)
Author: dsanders
Date: Tue Sep 15 08:46:21 2015
New Revision: 247686
URL: http://llvm.org/viewvc/llvm-project?rev=247686&view=rev
Log:
Revert r247684 - Replace Triple with a new TargetTuple ...
LLDB needs to be updated in the same commit.
Modified:
cfe/trunk/lib/Parse/ParseStmtAsm.cpp
c
Author: dsanders
Date: Tue Sep 15 09:08:28 2015
New Revision: 247692
URL: http://llvm.org/viewvc/llvm-project?rev=247692&view=rev
Log:
Re-commit r247683: Replace Triple with a new TargetTuple in MCTargetDesc/* and
related. NFC.
Summary:
This is the first patch in the series to migrate Triple's (
Author: dsanders
Date: Tue Sep 15 11:17:27 2015
New Revision: 247702
URL: http://llvm.org/viewvc/llvm-project?rev=247702&view=rev
Log:
Revert r247692: Replace Triple with a new TargetTuple in MCTargetDesc/* and
related. NFC.
Eric has replied and has demanded the patch be reverted.
Modified:
dsanders accepted this revision.
dsanders added a comment.
This revision is now accepted and ready to land.
LGTM
It's not for this patch but I think we need a general solution to the -Wa,...
options. It seems wrong to implement each assembler option twice (both with and
without the '-Wa,').
h
Author: dsanders
Date: Tue Sep 22 08:52:32 2015
New Revision: 248276
URL: http://llvm.org/viewvc/llvm-project?rev=248276&view=rev
Log:
[mips] Added support for using the command line options -Wa,-msoft-float and
-Wa,-mhard-float.
Patch by Scott Egerton.
Reviewers: vkalintiris, dsanders
Subscri
dsanders created this revision.
dsanders added a reviewer: atanasyan.
dsanders added a subscriber: cfe-commits.
Strictly speaking, the MIPS*R2 ISA's should not permit -mnan=2008 since this
feature was added in MIPS*R3. However, other toolchains permit this and we
should do the same.
http://review
Author: dsanders
Date: Thu Sep 24 05:22:17 2015
New Revision: 248481
URL: http://llvm.org/viewvc/llvm-project?rev=248481&view=rev
Log:
[mips] Relax -mnan=2008 acceptance to permit MIPS32R2 and MIPS64R2.
Summary:
Strictly speaking, the MIPS*R2 ISA's should not permit -mnan=2008 since this
feature
dsanders added subscribers: rengolin, joerg.
dsanders added a comment.
+Renato and Joerg
I was going to say I think it's ok and the optimizer should be smart enough to
factor out the common IsMips check but I've just realized there may be a better
way. The current code is using an else after an
so
> that they are not checked on other platforms.
>
> On Thu, Sep 24, 2015 at 10:22:29AM +, Daniel Sanders via cfe-commits
> wrote:
> > I'm thinking something like:
>
> I think we really want to have an outer case, platform specific -Wa
> options are quite c
Author: dsanders
Date: Mon Oct 5 07:24:30 2015
New Revision: 249306
URL: http://llvm.org/viewvc/llvm-project?rev=249306&view=rev
Log:
[mips][p5600] Add -mcpu=p5600 option.
Summary:
Reviewers: vkalintiris, atanasyan
Subscribers: cfe-commits
Differential Revision: http://reviews.llvm.org/D12234
dsanders accepted this revision.
dsanders added a comment.
This revision is now accepted and ready to land.
LGTM
http://reviews.llvm.org/D13100
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cf
dsanders created this revision.
dsanders added reviewers: vkalintiris, atanasyan.
dsanders added a subscriber: cfe-commits.
dsanders added a dependency: D12193: [mips][p5600] Added P5600 processor and
initial scheduler..
Depends on D12193
http://reviews.llvm.org/D12234
Files:
lib/Basic/Target
Author: dsanders
Date: Thu Sep 3 07:58:39 2015
New Revision: 246765
URL: http://llvm.org/viewvc/llvm-project?rev=246765&view=rev
Log:
[mips] Added support for choosing between traps and breaks in the integrated
assembler macros.
Summary: The command line options for these are -Wa,--trap and -Wa
dsanders accepted this revision.
dsanders added a comment.
This revision is now accepted and ready to land.
LGTM with a few nits
Comment at: include/clang/Basic/Attr.td:841-842
@@ +840,4 @@
+def MipsInterrupt : InheritableAttr, TargetSpecificAttr {
+ // NOTE: if you add any add
Author: dsanders
Date: Tue Oct 27 13:04:42 2015
New Revision: 251430
URL: http://llvm.org/viewvc/llvm-project?rev=251430&view=rev
Log:
[mips] Separated mips specific -Wa options, so that they are not checked on
other platforms.
Summary: This is a follow on to post review comments on revision r24
Author: dsanders
Date: Tue Jul 19 05:49:03 2016
New Revision: 275967
URL: http://llvm.org/viewvc/llvm-project?rev=275967&view=rev
Log:
[mips] Correct label prefixes for N32 and N64.
Summary:
N32 and N64 follow the standard ELF conventions (.L) whereas O32 uses its own
($).
This fixes the majorit
dsanders updated this revision to Diff 64870.
dsanders added a comment.
Refresh and ping
https://reviews.llvm.org/D21070
Files:
lib/Basic/Targets.cpp
lib/Driver/ToolChains.cpp
lib/Driver/Tools.cpp
lib/Frontend/CompilerInvocation.cpp
tools/driver/cc1as_main.cpp
Index: tools/driver/cc1
dsanders created this revision.
dsanders added a reviewer: sdardis.
dsanders added a subscriber: cfe-commits.
Unfortunately we can't enable it for all N64 because it is not yet possible to
distinguish N32 from N64 from the triple on other environments.
Thanks for reverting this, our internal buildbots have gone green again.
> Can you add a non-regression test case?
> (It seems the test-suite didn’t catch this bug right?)
Do we have any upstream buildbots using -O0? I haven't investigated very far*
but -O0 seems to be the common factor between
dsanders updated this revision to Diff 65918.
dsanders added a comment.
Refresh and ping
https://reviews.llvm.org/D21070
Files:
lib/Basic/Targets.cpp
lib/Driver/ToolChains.cpp
lib/Driver/Tools.cpp
lib/Frontend/CompilerInvocation.cpp
tools/driver/cc1as_main.cpp
Index: tools/driver/cc1
dsanders added a comment.
One last ping since I need to either commit this series next week or hand over
to a colleague to continue it.
https://reviews.llvm.org/D21070
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cg
dsanders added a comment.
One last ping since I need to either commit this series next week or hand over
to a colleague to continue it.
https://reviews.llvm.org/D21072
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cg
dsanders added a comment.
FWIW, the changes in the last revision look minor to me. I doubt it affects the
LGTM
> Do we have any test cases for arbitrary sized trivally copyable structures?
> That might also be needed.
The failures in 3.8.0rc1 (and presumably still occur in the 3.8.0rc2 that w
dsanders accepted this revision.
dsanders added a comment.
LGTM
http://reviews.llvm.org/D16613
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Hi,
In my latests rc2+patches build I've also found that we need to rename
HAVE_CXX_ATOMICS_WITH_LIB to something like LIBCXX_HAVE_CXX_ATOMICS_WITH_LIB.
It's currently re-using the result of LLVM's check which doesn't include 64-bit
atomics.
From: Vasile
dsanders created this revision.
dsanders added reviewers: mclow.lists, hans.
dsanders added a subscriber: cfe-commits.
On glibc, the bits used for the various character classes is endian dependant
(see _ISbit() in ctypes.h) but __regex_word does not account for this and uses
a spare bit that isn't
dsanders added inline comments.
Comment at: include/regex:980
@@ +979,3 @@
+#if defined(__GLIBC__)
+static const char_class_type __regex_word =
static_cast(_ISbit(15));
+#else
The static_cast is necessary to silence a false-positive warning on
little-endian.
dsanders added a comment.
In http://reviews.llvm.org/D17132#349993, @mclow.lists wrote:
> In r260527, I added some tests to catch this if it happens again.
>
> If those tests fail w/o this patch and succeed with, then I'm happy with
> applying it.
The tests in r260527, don't fail without this
dsanders added a comment.
I haven't fully proven this yet (because I haven't found the table), but I
think that C and C++ are sharing the same table and we're colliding with a bit
that only makes sense to C. This would explain why your tests don't fail but
regex_word still collides with _ISgrap
dsanders added a comment.
The table is apparently inside locale-archive and it looks like C and C++ both
share this table. At this point I'm satisfied that the table also contains
information that C++ isn't interested in and it's this data that regex_word
collides with.
Are you happy to lift t
e a cmake module to figure out whether
we need to link with libatomic."
- [libcxx] r260524 - Fix r260515 - Correct typos in CMake changes
- [libcxx] r260531 - Rename CheckLibcxxAtomic.cmake variable result names so
they don't clash with LLVM
@Marshall Any objections?
On Thu, Feb 11,
dsanders added a comment.
Sorry for the early ping but I need to fix this for rc3 (which Hans is hoping
to tag mid-week) and I'm stuck as long as the requirement that the C++ tests in
r260527 fail without my patch is in place.
I'm happy to make this '#ifdef __mips__' if that helps.
http://rev
_LIB to something like
>>>> LIBCXX_HAVE_CXX_ATOMICS_WITH_LIB.
>>> Fixed in r260531.
>>>
>>> I think we will eventually want to merge the following commits, assuming
>>> they don't regress the build, especially with the -gcc-toolchain o
Thanks. I'll commit using a MIPS ifdef in the morning and merge once it's
through the buildbots.
From: Marshall Clow [mclow.li...@gmail.com]
Sent: 16 February 2016 22:37
To: Daniel Sanders; h...@chromium.org; mclow.li...@gmail.com
Cc: cfe-commits@lists.llvm
Author: dsanders
Date: Wed Feb 17 07:16:31 2016
New Revision: 261088
URL: http://llvm.org/viewvc/llvm-project?rev=261088&view=rev
Log:
[libcxx] Fix definition of regex_traits::__regex_word on big-endian glibc
systems
Summary:
On glibc, the bits used for the various character classes is endian de
Author: dsanders
Date: Wed Feb 17 09:02:33 2016
New Revision: 261097
URL: http://llvm.org/viewvc/llvm-project?rev=261097&view=rev
Log:
Merging r261088:
r261088 | dsanders | 2016-02-17 13:16:31 + (Wed, 17 Feb 2016) | 21 li
dsanders added a comment.
Can you explain what the problem was and why this change is needed?
I'm guessing it's something to do with the 'Features[CPU] = true' line.
Repository:
rL LLVM
http://reviews.llvm.org/D16139
___
cfe-commits mailing list
Hi,
This change makes sense but it needs a test case.
> -Original Message-
> From: cfe-commits [mailto:cfe-commits-boun...@lists.llvm.org] On Behalf
> Of Vasileios Kalintiris via cfe-commits
> Sent: 13 May 2016 13:13
> To: cfe-commits@lists.llvm.org
> Subject: r269411 - [mips] Consult tri
Author: dsanders
Date: Sat May 14 07:43:08 2016
New Revision: 269560
URL: http://llvm.org/viewvc/llvm-project?rev=269560&view=rev
Log:
[mips] Enable IAS by default for 32-bit MIPS targets (O32).
Summary:
The MIPS IAS can now pass 'ninja check-all', recurse, build a bootable linux
kernel, and pass
dsanders added a subscriber: dsanders.
Comment at: test/Preprocessor/init.c:3295
@@ +3294,3 @@
+// MIPSN32BE: #define __GCC_HAVE_SYNC_COMPARE_AND_SWAP_4 1
+// MIPSN32BE: #define __GCC_HAVE_SYNC_COMPARE_AND_SWAP_8 1
+// MIPSN32BE-NOT: __GCC_HAVE_SYNC_COMPARE_AND_SWAP_16
---
dsanders created this revision.
dsanders added a reviewer: vkalintiris.
dsanders added a subscriber: cfe-commits.
dsanders added a dependency: D18753: [mips][sanitizer_common] Don't use `ld` in
internal_clone() on 32-bit MIPS..
The MIPS IAS can now pass 'ninja check-all' and recurse now that the
dsanders created this revision.
dsanders added subscribers: cfe-commits, atanasyan.
Herald added subscribers: dschuff, jfb.
This unifies mips/mipsel and mips64/mips64el into a single class so that we can
later support O32 on mips64/mips64el and N32/N64 on mips/mipsel (when an
appropriate CPU selec
dsanders created this revision.
dsanders added subscribers: cfe-commits, atanasyan.
dsanders added a dependency: D20678: [mips] Fold MipsTargetInfoBase subclasses
into MipsTargetInfoBase and rename to MipsTargetInfo. NFC.
There are no clang or llvm* tests for EABI and no EABI buildbots.
*There i
dsanders created this revision.
dsanders added subscribers: cfe-commits, atanasyan.
dsanders added a dependency: D20679: [mips] Kill 'support' for untested EABI..
Depends on D20679
http://reviews.llvm.org/D20680
Files:
lib/Basic/Targets.cpp
Index: lib/Basic/Targets.cpp
===
dsanders added inline comments.
Comment at: lib/Basic/Targets.cpp:6986
@@ +6985,3 @@
+ : TargetInfo(Triple), CPU((getTriple().getArch() == llvm::Triple::mips ||
+ getTriple().getArch() == llvm::Triple::mipsel)
+
Hi,
I think this commit may have caused the failure in
http://lab.llvm.org:8011/builders/clang-cmake-mips/builds/13743. Could you
check? Buildbot will have supressed the usual email because the previous build
failed (for a different reason).
> -Original Message-
> From: cfe-commits [ma
Thanks. r270962 seems to have fixed the buildbot.
> -Original Message-
> From: Daniel Sanders
> Sent: 26 May 2016 16:24
> To: 'Alexey Bataev'; cfe-commits@lists.llvm.org
> Subject: RE: r270838 - [OPENMP] Add option '-fopenmp-version=[31|40|45]'
> allowing choosing
>
> Hi,
>
> I think thi
Author: dsanders
Date: Fri May 27 06:51:02 2016
New Revision: 270984
URL: http://llvm.org/viewvc/llvm-project?rev=270984&view=rev
Log:
[mips] Fold MipsTargetInfoBase subclasses into MipsTargetInfoBase and rename to
MipsTargetInfo. NFC
Summary:
This unifies mips/mipsel and mips64/mips64el into a
This revision was automatically updated to reflect the committed changes.
Closed by commit rL270984: [mips] Fold MipsTargetInfoBase subclasses into
MipsTargetInfoBase and rename… (authored by dsanders).
Changed prior to commit:
http://reviews.llvm.org/D20678?vs=58604&id=58770#toc
Repository:
dsanders updated this revision to Diff 58772.
dsanders added a comment.
Herald added subscribers: sdardis, emaste.
Removed the 'eabi' from Tools.cpp this revealed three driver tests for passing
-mabi to the assembler but the ABI itself is still completely untested.
http://reviews.llvm.org/D20679
dsanders added a comment.
Just to double-check: Do you still agree we should kill this now that the
number of tests covering EABI-specific behaviour is non-zero?
I think we should on the basis that we still have no backend tests and no
buildbots. The only tests we have check that the driver pas
Author: dsanders
Date: Fri May 27 09:30:23 2016
New Revision: 270998
URL: http://llvm.org/viewvc/llvm-project?rev=270998&view=rev
Log:
[mips] Kill 'support' for untested EABI.
Summary:
There are no llvm backend tests* for EABI and no EABI buildbots. There were only
three clang tests, all of which
This revision was automatically updated to reflect the committed changes.
Closed by commit rL270998: [mips] Kill 'support' for untested EABI. (authored
by dsanders).
Changed prior to commit:
http://reviews.llvm.org/D20679?vs=58772&id=58782#toc
Repository:
rL LLVM
http://reviews.llvm.org/D20
Author: dsanders
Date: Fri Jun 3 05:11:01 2016
New Revision: 271647
URL: http://llvm.org/viewvc/llvm-project?rev=271647&view=rev
Log:
[mips] Slightly simplify MipsTargetInfo::setDataLayout(). NFC.
Summary:
Reviewers: atanasyan
Subscribers: atanasyan, cfe-commits
Differential Revision: http://
dsanders created this revision.
dsanders added a reviewer: atanasyan.
dsanders added a subscriber: cfe-commits.
setABI() is still tied to the Arch component of the Triple to preserve existing
behaviour.
http://reviews.llvm.org/D20961
Files:
lib/Basic/Targets.cpp
Index: lib/Basic/Targets.cpp
=
dsanders created this revision.
dsanders added a reviewer: atanasyan.
dsanders added a subscriber: cfe-commits.
dsanders added a dependency: D20961: [mips] Replace almost all Arch checks in
MipsTargetInfo with ABI checks. NFC..
Herald added a subscriber: sdardis.
Depends on D20961
http://reviews
Author: dsanders
Date: Mon Jun 6 04:07:08 2016
New Revision: 271875
URL: http://llvm.org/viewvc/llvm-project?rev=271875&view=rev
Log:
[mips] Replace almost all Arch checks in MipsTargetInfo with ABI checks. NFC.
Summary:
setABI() is still tied to the Arch component of the Triple to preserve exis
Author: dsanders
Date: Mon Jun 6 04:47:32 2016
New Revision: 271877
URL: http://llvm.org/viewvc/llvm-project?rev=271877&view=rev
Log:
[mips] The P5600 does not support N32/N64 since it's a 32-bit CPU.
Summary:
Reviewers: atanasyan
Subscribers: cfe-commits, sdardis
Differential Revision: http:
dsanders created this revision.
dsanders added a reviewer: atanasyan.
dsanders added a subscriber: cfe-commits.
Herald added a subscriber: sdardis.
32-bit CPU's default to O32. 64-bit CPU's default to N64. The default CPU
(mips32r2/mips64r2) still depends on the arch so there's no functional
chang
Author: dsanders
Date: Mon Jun 6 07:02:21 2016
New Revision: 271884
URL: http://llvm.org/viewvc/llvm-project?rev=271884&view=rev
Log:
[mips] The default ABI depends on the CPU not the Arch on MTI and IMG vendor
triples.
Summary:
32-bit CPU's default to O32. 64-bit CPU's default to N64. The defa
dsanders created this revision.
dsanders added a reviewer: atanasyan.
dsanders added a subscriber: cfe-commits.
Herald added a subscriber: sdardis.
The validity of ABI/CPU pairs is no longer checked on the fly but is
instead checked after initialization. As a result, invalid CPU/ABI pairs
can be r
dsanders closed this revision.
dsanders added a comment.
This has already been committed. I'm not sure why it didn't auto-close.
http://reviews.llvm.org/D18761
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mai
dsanders created this revision.
dsanders added a subscriber: cfe-commits.
dsanders added a dependency: D21069: [mips] Require that ABI's are passed in
the triple within LLVM..
Herald added subscribers: srhines, danalbert, tberghammer.
'clang -cc1' and 'clang -cc1as' will mutate the triple to acco
dsanders created this revision.
dsanders added a subscriber: cfe-commits.
dsanders added a dependency: D21070: Pass the ABI in the triple when
appropriate (currently for MIPS) for 'clang -cc1' and 'clang -cc1as'.
Herald added subscribers: dschuff, jfb.
This stops cases such as '-target mips-linux
dsanders abandoned this revision.
dsanders added a comment.
Part of this patch is missing. I'll repost it soon.
http://reviews.llvm.org/D21071
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe
dsanders created this revision.
dsanders added subscribers: jfb, dschuff, cfe-commits.
dsanders added a dependency: D21070: Pass the ABI in the triple when
appropriate (currently for MIPS) for 'clang -cc1' and 'clang -cc1as'.
Herald added subscribers: sdardis, srhines, danalbert, tberghammer.
Thi
Author: dsanders
Date: Tue Jun 14 03:58:50 2016
New Revision: 272645
URL: http://llvm.org/viewvc/llvm-project?rev=272645&view=rev
Log:
[mips] Defer validity check for CPU/ABI pairs and improve error message for
invalid cases.
Summary:
The validity of ABI/CPU pairs is no longer checked on the fly
Author: dsanders
Date: Thu Jun 23 04:29:38 2016
New Revision: 273552
URL: http://llvm.org/viewvc/llvm-project?rev=273552&view=rev
Log:
Attempt to fix MIPS buildbots after r273425.
MIPS has a 'signext' attribute that was causing the check to fail.
Modified:
cfe/trunk/test/CodeGenOpenCL/kernel
dsanders added a comment.
This change agrees with what I think the calling convention is and the
documentation. However, I've hit quite a few discrepancies between the
documented calling convention and the de-facto one implemented by gcc so I'm
wary of going by that alone.
Have you tried putti
dsanders accepted this revision.
dsanders added a comment.
This revision is now accepted and ready to land.
In that case the MIPS side of this LGTM. Someone more familiar with ARM should
approve it for ARM.
http://reviews.llvm.org/D21611
___
cfe-co
dsanders added a comment.
What sets HAVE_CXX_LIBATOMICS64? I don't see a reference to LLVM's CheckAtomic.
Repository:
rL LLVM
http://reviews.llvm.org/D21599
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mai
dsanders added a comment.
Have you tried a standalone build? I suspect HAVE_CXX_LIBATOMICS64 will never
be set for that case. If that's the case, adding 'include(CheckAtomic)' will
probably fix it.
Repository:
rL LLVM
http://reviews.llvm.org/D21599
___
dsanders added a comment.
I haven't used it that way either and I'm not sure it's a supported build for
libcxx but most LLVM projects support a standalone build as far as I know. Can
someone from the libcxx project confirm whether standalone builds are supported?
Repository:
rL LLVM
http://
dsanders updated this revision to Diff 63676.
dsanders added a comment.
Updated to match latest version of http://reviews.llvm.org/D21467.
At this point Mips is in the X86/everyone-else camp but there's a loose end to
tie up in LLVM in order to prove it. Once LLVM rejects non-empty ABIName's
for
Author: dsanders
Date: Tue Mar 1 11:15:11 2016
New Revision: 262350
URL: http://llvm.org/viewvc/llvm-project?rev=262350&view=rev
Log:
Explicitly select IAS in new embed-bitcode.c test.
This should fix clang-cmake-mips builder since MIPS does not have IAS enabled
by default (yet).
Modified:
Hi,
I've made a small change to the test case in r262350 to fix the
clang-cmake-mips builder. MIPS doesn't
enable the integrated assembler by default yet so it was failing to find the
-emit-obj option. I've fixed this by adding -fintegrated-as.
From: cfe
Author: dsanders
Date: Tue Mar 1 15:57:22 2016
New Revision: 262409
URL: http://llvm.org/viewvc/llvm-project?rev=262409&view=rev
Log:
Explicitly select IAS on one more RUN line in new embed-bitcode.c test.
This should fix clang-cmake-mips builder since MIPS does not have IAS enabled
by default (
No problem.
It seems I missed one of the relevant RUN lines on the first attempt. I've
fixed that in r262409.
From: steve...@apple.com [steve...@apple.com]
Sent: 01 March 2016 17:26
To: Daniel Sanders
Cc: cfe-commits@lists.llvm.org
Subject: Re: r262282 -
dsanders accepted this revision.
dsanders added a comment.
Thanks. LGTM
Repository:
rL LLVM
http://reviews.llvm.org/D16139
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
dsanders added a comment.
In http://reviews.llvm.org/D16139#368217, @echristo wrote:
> This seems wrong. You should fix setCPU instead or set a default CPU.
We already set a default CPU in the constructor (e.g.
Mips32TargetInfoBase::Mips32TargetInfoBase() provides "mips32r2"). It's the CPU
ar
> > From: Eric Christopher [echri...@gmail.com]
> > Sent: 09 March 2016 06:50
> > To: reviews+d16139+public+275805419034a...@reviews.llvm.org; Bhushan
> > Attarde; Vasileios Kalintiris; Daniel Sanders
> > Cc: Sagar Thakur; Nitesh Jain; Mohit Bhakkad; Jaydeep Patil;
> > cfe-commits@lists.llvm.org
dsanders added a comment.
Thanks for working on this.
Comment at: llvm/lib/Target/Mips/AsmParser/MipsAsmParser.cpp:2715-2718
@@ -2714,3 +2714,6 @@
- unsigned TrgReg;
+ // TrgReg should never normally be assigned NUM_TARGET_REGS.
+ // If you end up with NUM_TARGET_REGS, you
dsanders added subscribers: eugenis, ygorshenin.
dsanders added a comment.
I think it refers to MCTargetOptions::SanitizeAddress but I don't know where
we'd get that information from. Unfortunately, it seems that the original
author (@ygorshenin) might not be on the list anymore (phabricator sho
dsanders added a comment.
> > > b) CPUs are not subtarget features (or they shouldn't be), they're CPUs
> > > that contain features. They may be generic names for ISAs as well, but
> > > probably best to keep them separate.
> >
> > I agree, we have two separate concepts that happen to use t
dsanders accepted this revision.
dsanders added a comment.
This revision is now accepted and ready to land.
LGTM
Sorry for missing this when it was first posted. I've also been away for
roughly half of the two weeks since Petar added me to the review.
http://reviews.llvm.org/D17378
Did you get an answer to this question? I think it's just that the FreeBSD
class hasn't refactored to that style yet.
That switch statement is getting quite large so it would be a nice cleanup to
switch to that style.
> -Original Message-
> From: cfe-commits [mailto:cfe-commits-boun...@l
-G isn't handled there at the moment but ClangAs::AddMIPSTargetArgs() and
Clang::AddMIPSTargetArgs() would be the right place to add it in the Linux
toolchain when we add it.
The cleanup I'm referring to is that someone hoisted the bulk of the code out
of the big switch-statement in Clang::Cons
Hi Chris,
This doesn't seem to work when building clang separately from llvm. LLVMLinux
fails to build clang with:
CMake Error at CMakeLists.txt:104 (include):
include could not find load file:
VersionFromVCS
CMake Error at CMakeLists.txt:222 (add_v
Thanks, that did the trick. I've removed the workaround from LLVMLinux.
> -Original Message-
> From: cbiene...@apple.com [mailto:cbiene...@apple.com] On Behalf Of
> Chris Bieneman
> Sent: 15 January 2016 17:55
> To: Daniel Sanders
> Cc: cfe-commits@lists.llvm.org
> Subject: Re: r257827 - [
dsanders created this revision.
dsanders added reviewers: mclow.lists, hans.
dsanders added a subscriber: cfe-commits.
http://reviews.llvm.org/D16406
Files:
test/std/input.output/file.streams/fstreams/filebuf.virtuals/overflow.pass.cpp
test/std/input.output/file.streams/fstreams/filebuf.virt
dsanders added a comment.
Hi,
I'd like to merge this to the 3.8 branch once it has been accepted.
http://reviews.llvm.org/D16406
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
dsanders updated this revision to Diff 45531.
dsanders added a comment.
Added one more. It was also guarded by a check for ru_RU.UTF-8 so it was missed
on the first sweep.
http://reviews.llvm.org/D16406
Files:
test/std/input.output/file.streams/fstreams/filebuf.virtuals/overflow.pass.cpp
dsanders added a comment.
Thanks. I've added one more which came up after enabling all the missing
locales except for en_US.UTF-8.
I'll commit this and find the other 8 with grep afterwards.
http://reviews.llvm.org/D16406
___
cfe-commits mailing l
1 - 100 of 125 matches
Mail list logo
