Benjamin Kramer [Thu, 28 May 2015 11:45:32 +0000 (11:45 +0000)]
[MC] Replace custom string join function with the one from StringExtras.
NFC.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238414
91177308-0d34-0410-b5e6-
96231b3b80d8
Benjamin Kramer [Thu, 28 May 2015 11:24:24 +0000 (11:24 +0000)]
Don't call utostr in Twine/raw_ostream contexts.
Creating temporary std::strings there is unnecessary.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238412
91177308-0d34-0410-b5e6-
96231b3b80d8
Renato Golin [Thu, 28 May 2015 08:59:03 +0000 (08:59 +0000)]
Fix comments in ARMTargetParser. NFC.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238404
91177308-0d34-0410-b5e6-
96231b3b80d8
Yury Gribov [Thu, 28 May 2015 08:03:28 +0000 (08:03 +0000)]
[ASan] Fix previous commit. Patch by Max Ostapenko!
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238403
91177308-0d34-0410-b5e6-
96231b3b80d8
Yury Gribov [Thu, 28 May 2015 07:51:49 +0000 (07:51 +0000)]
[ASan] New approach to dynamic allocas unpoisoning. Patch by Max Ostapenko!
Differential Revision: http://reviews.llvm.org/D7098
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238402
91177308-0d34-0410-b5e6-
96231b3b80d8
Craig Topper [Thu, 28 May 2015 06:38:32 +0000 (06:38 +0000)]
[TableGen] Use DefInit::getAsString instead of getDef()->getName(). NFC
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238399
91177308-0d34-0410-b5e6-
96231b3b80d8
Craig Topper [Thu, 28 May 2015 06:38:28 +0000 (06:38 +0000)]
[TableGen] Don't convert types to strings to query what they are. Just use 'isa'
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238398
91177308-0d34-0410-b5e6-
96231b3b80d8
David Majnemer [Thu, 28 May 2015 06:16:39 +0000 (06:16 +0000)]
[Reassociate] Canonicalizing 'x [+-] (-Constant * y)' isn't always a win
Canonicalizing 'x [+-] (-Constant * y)' is not a win if we don't *know*
we will open up CSE opportunities.
If the multiply was 'nsw', then negating 'y' requires us to clear the
'nsw' flag. If this is actually worth pursuing, it is probably more
appropriate to do so in GVN or EarlyCSE.
This fixes PR23675.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238397
91177308-0d34-0410-b5e6-
96231b3b80d8
Jingyue Wu [Thu, 28 May 2015 04:56:52 +0000 (04:56 +0000)]
[NaryReassociate] Run EarlyCSE after NaryReassociate
Summary:
This patch made two improvements to NaryReassociate and the NVPTX pipeline
1. Run EarlyCSE/GVN after NaryReassociate to get rid of redundant common
expressions.
2. When adding an instruction to SeenExprs, maps both the SCEV before and after
reassociation to that instruction.
Test Plan: updated @reassociate_gep_nsw in nary-gep.ll
Reviewers: meheff, broune
Reviewed By: broune
Subscribers: dberlin, jholewinski, llvm-commits
Differential Revision: http://reviews.llvm.org/D9947
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238396
91177308-0d34-0410-b5e6-
96231b3b80d8
Chandler Carruth [Thu, 28 May 2015 02:40:15 +0000 (02:40 +0000)]
[x86] Refactor the tests for popcnt.
Extracted from the D6531 patch by Bruno Cardoso Lopes, and re-generated
to reflect the current state of the world. This should let Bruno's D6531
actually show the delta between the approaches by running the x86 test
case update script after re-building.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238391
91177308-0d34-0410-b5e6-
96231b3b80d8
Chandler Carruth [Thu, 28 May 2015 02:17:15 +0000 (02:17 +0000)]
[omp] Actually provide a default OpenMP runtime -- libgomp for now.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238390
91177308-0d34-0410-b5e6-
96231b3b80d8
Chandler Carruth [Thu, 28 May 2015 01:47:22 +0000 (01:47 +0000)]
[omp] Add a configuration variable for the default OpenMP runtime.
This will be used in my next commit to Clang.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238388
91177308-0d34-0410-b5e6-
96231b3b80d8
Duncan P. N. Exon Smith [Wed, 27 May 2015 23:02:36 +0000 (23:02 +0000)]
AsmPrinter: Avoid a warning in NDEBUG, NFC
Should fix the -Werror release build:
http://lab.llvm.org:8011/builders/lld-x86_64-darwin13/builds/11113
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238375
91177308-0d34-0410-b5e6-
96231b3b80d8
Duncan P. N. Exon Smith [Wed, 27 May 2015 22:59:03 +0000 (22:59 +0000)]
AsmPrinter: Return added DIE from DIE::addChild()
Change `DIE::addChild()` to return a reference to the just-added node,
and update consumers to use it directly. An upcoming commit will
abstract away (and eventually change) the underlying storage of
`DIE::Children`.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238372
91177308-0d34-0410-b5e6-
96231b3b80d8
Fiona Glaser [Wed, 27 May 2015 22:51:47 +0000 (22:51 +0000)]
RegisterPressure: fix debug prints in case of physical registers
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238371
91177308-0d34-0410-b5e6-
96231b3b80d8
Duncan P. N. Exon Smith [Wed, 27 May 2015 22:44:06 +0000 (22:44 +0000)]
AsmPrinter: Stop exposing underlying DIEValue list, NFC
Change the `DIE` API to hide the implementation of the list of
`DIEValue`s.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238369
91177308-0d34-0410-b5e6-
96231b3b80d8
Duncan P. N. Exon Smith [Wed, 27 May 2015 22:36:37 +0000 (22:36 +0000)]
AsmPrinter: Remove DIEHash::AttrEntry, NFC
Remove "the most boring struct ever" (thanks to review by Eric).
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238366
91177308-0d34-0410-b5e6-
96231b3b80d8
Duncan P. N. Exon Smith [Wed, 27 May 2015 22:31:41 +0000 (22:31 +0000)]
AsmPrinter: Store abbreviation data directly in DIE and DIEValue
Stop storing a `DIEAbbrev` in `DIE`, since the data fits neatly inside
the `DIEValue` list. Besides being a cleaner data structure (avoiding
the parallel arrays), this gives us more freedom to rearrange the
`DIEValue` list.
This fixes the temporary memory regression from 845 MB up to 879 MB, and
drops it further to 829 MB for a net memory decrease of around 1.9%
(incremental decrease around 5.7%).
(I'm looking at `llc` memory usage on `verify-uselistorder.lto.opt.bc`;
see r236629 for details.)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238364
91177308-0d34-0410-b5e6-
96231b3b80d8
Duncan P. N. Exon Smith [Wed, 27 May 2015 22:14:58 +0000 (22:14 +0000)]
Reapply "AsmPrinter: Change DIEValue to be stored by value"
This reverts commit r238350, effectively reapplying r238349 after fixing
(all?) the problems, all somehow related to how I was using
`AlignedArrayCharUnion<>` inside `DIEValue`:
- MSVC can only handle `sizeof()` on types, not values. Change the
assert.
- GCC doesn't know the `is_trivially_copyable` type trait. Instead of
asserting it, add destructors.
- Call placement new even when constructing POD (i.e., the pointers).
- Instead of copying the char buffer, copy the casted classes.
I've left in a couple of `static_assert`s that I think both MSVC and GCC
know how to handle. If the bots disagree with me, I'll remove them.
- Check that the constructed type is either standard layout or a
pointer. This protects against a programming error: we really want
the "small" `DIEValue`s to be small and simple, so don't
accidentally change them not to be.
- Similarly, check that the size of the buffer is no bigger than a
`uint64_t` or a pointer. (I thought checking against
`sizeof(uint64_t)` would be good enough, but Chandler suggested that
pointers might sometimes be bigger than that in the context of
sanitizers.)
I've also committed r238359 in the meantime, which introduces a
DIEValue.def to simplify dispatching between the various types (thanks
to a review comment by David Blaikie). Without that, this commit would
be almost unintelligible.
Here's the original commit message:
--
Change `DIEValue` to be stored/passed/etc. by value, instead of
reference. It's now a discriminated union, with a `Val` field storing
the actual type. The classes that used to inherit from `DIEValue` no
longer do. There are two categories of these:
- Small values fit in a single pointer and are stored by value.
- Large values require auxiliary storage, and are stored by reference.
The only non-mechanical change is to tools/dsymutil/DwarfLinker.cpp. It
was relying on `DIEInteger`s being passed around by reference, so I
replaced that assumption with a `PatchLocation` type that stores a safe
reference to where the `DIEInteger` lives instead.
This commit causes a temporary regression in memory usage, since I've
left merging `DIEAbbrevData` into `DIEValue` for a follow-up commit. I
measured an increase from 845 MB to 879 MB, around 3.9%. The follow-up
drops it lower than the starting point, and I've only recently brought
the memory this low anyway, so I'm committing these changes separately
to keep them incremental. (I also considered swapping the commits, but
the other one first would cause a lot more code churn.)
(I'm looking at `llc` memory usage on `verify-uselistorder.lto.opt.bc`;
see r236629 for details.)
--
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238362
91177308-0d34-0410-b5e6-
96231b3b80d8
Duncan P. N. Exon Smith [Wed, 27 May 2015 21:15:43 +0000 (21:15 +0000)]
AsmPrinter: Introduce DIEValue.def, NFC
Use a .def macro file to iterate through the various subclasses of
`DIEValue`.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238359
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Wed, 27 May 2015 21:04:14 +0000 (21:04 +0000)]
Rename and move getCurrentSectionData.
I think this concludes the merge of MCSectionData and MCSection.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238358
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Wed, 27 May 2015 20:52:32 +0000 (20:52 +0000)]
There is only one current section.
Both MCStreamer and MCObjectStreamer were maintaining a current section
variable and they were slightly out of sync. I don't think this was observable,
but was inefficient and error prone.
Changing this requires a few cascading changes:
* SwitchSection has to call ChangeSection earlier for ChangeSection to see
the old section.
* With that change, ChangeSection cannot call EmitLabel, since during
ChangeSection we are still in the old section.
* When the object streamer requires a begin label, just reused the existing
generic support for begin labels instead of calling EmitLabel directly.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238357
91177308-0d34-0410-b5e6-
96231b3b80d8
Lang Hames [Wed, 27 May 2015 20:50:01 +0000 (20:50 +0000)]
[RuntimeDyld] Fix MachO i386 SECTDIFF relocation to support non-zero addends.
Previously, relocations of the form 'A - B + C' would fail on i386 when C was
non-zero.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238356
91177308-0d34-0410-b5e6-
96231b3b80d8
Renato Golin [Wed, 27 May 2015 19:49:53 +0000 (19:49 +0000)]
ARMTargetParser: Make BSD Thumb/BE armv6 work
Simple change to make arch like "thumbv6" and "armbev6" to return the
correct CPU for FreeBSD and NetBSD.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238353
91177308-0d34-0410-b5e6-
96231b3b80d8
Alex Lorenz [Wed, 27 May 2015 19:39:44 +0000 (19:39 +0000)]
MIR: Remove 2 empty non-reverted directories form SVN.
The revert commit r238007 didn't delete the old 'MIR' directories
in 'include/llvm/CodeGen' and 'lib/CodeGen'. Since the resubmitted
commit r238341 uses the 'MIRParser' directories instead of 'MIR'
this commit deletes the unused directories.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238352
91177308-0d34-0410-b5e6-
96231b3b80d8
Diego Novillo [Wed, 27 May 2015 19:34:01 +0000 (19:34 +0000)]
Final fix for PR 23499 and IR test case.
This fixes a bit I forgot in r238335. In addition to the data record and
the counter, we can also move the name of the counter to the comdat for
the associated function.
I'm also adding an IR test case to check that these three elements are
placed in the proper comdat.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238351
91177308-0d34-0410-b5e6-
96231b3b80d8
Duncan P. N. Exon Smith [Wed, 27 May 2015 19:30:27 +0000 (19:30 +0000)]
Revert "AsmPrinter: Change DIEValue to be stored by value"
This reverts commit r238349, since it caused some errors on bots:
- std::is_trivially_copyable isn't available until GCC 5.0.
- It was complaining about strict aliasing with my use of
ArrayCharUnion.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238350
91177308-0d34-0410-b5e6-
96231b3b80d8
Duncan P. N. Exon Smith [Wed, 27 May 2015 19:22:50 +0000 (19:22 +0000)]
AsmPrinter: Change DIEValue to be stored by value
Change `DIEValue` to be stored/passed/etc. by value, instead of
reference. It's now a discriminated union, with a `Val` field storing
the actual type. The classes that used to inherit from `DIEValue` no
longer do. There are two categories of these:
- Small values fit in a single pointer and are stored by value.
- Large values require auxiliary storage, and are stored by reference.
The only non-mechanical change is to tools/dsymutil/DwarfLinker.cpp. It
was relying on `DIEInteger`s being passed around by reference, so I
replaced that assumption with a `PatchLocation` type that stores a safe
reference to where the `DIEInteger` lives instead.
This commit causes a temporary regression in memory usage, since I've
left merging `DIEAbbrevData` into `DIEValue` for a follow-up commit. I
measured an increase from 845 MB to 879 MB, around 3.9%. The follow-up
drops it lower than the starting point, and I've only recently brought
the memory this low anyway, so I'm committing these changes separately
to keep them incremental. (I also considered swapping the commits, but
the other one first would cause a lot more code churn.)
(I'm looking at `llc` memory usage on `verify-uselistorder.lto.opt.bc`;
see r236629 for details.)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238349
91177308-0d34-0410-b5e6-
96231b3b80d8
Duncan P. N. Exon Smith [Wed, 27 May 2015 18:49:38 +0000 (18:49 +0000)]
AsmPrinter: Reorganize DIE.h, NFC
An upcoming commit will switch to storing `DIEValue`s by value in a
`DIE`, so move the `DIE` down below all the subclasses it can. Two
subclasses inherit from `DIE`, so drop those down below `DIE`.
`DIEValue` will become a discriminated union, which contains an instance
of one of the classes that are currently subclasses. "Big" ones, such
as those that inherit from `DIE`, will continue to be stored by pointer.
This commit does as much of the code motion ahead of time as possible.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238346
91177308-0d34-0410-b5e6-
96231b3b80d8
Renato Golin [Wed, 27 May 2015 18:15:37 +0000 (18:15 +0000)]
ARMTargetParser: Normalising build attributes
Now that most of the methods in Clang and LLVM that were parsing arch/cpu/fpu
strings are using ARMTargetParser, it's time to make it a bit more conforming
with what the ABI says.
This commit adds some clarification on what build attributes are accepted and
which are "non-standard". It also makes clear that the "defaultCPU" and
"defaultArch" methods were really just build attribute getters.
It also diverges from GCC's behaviour to say that armv2/armv3 are really an
ARMv4 in the build attributes, when the ABI has a clear state for that: Pre-v4.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238344
91177308-0d34-0410-b5e6-
96231b3b80d8
Yaron Keren [Wed, 27 May 2015 18:11:07 +0000 (18:11 +0000)]
Avoid creating and destroying a std::string on every iteration.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238343
91177308-0d34-0410-b5e6-
96231b3b80d8
Alex Lorenz [Wed, 27 May 2015 18:02:19 +0000 (18:02 +0000)]
Resubmit r237954 (MIR Serialization: print and parse LLVM IR using MIR format).
This commit a 3rd attempt at comitting the initial MIR serialization patch.
The first commit (r237708) was reverted in 237730. Then the second commit
(r237954) was reverted in r238007, as the MIR library under CodeGen caused
a circular dependency where the CodeGen library depended on MIR and MIR
library depended on CodeGen.
This commit has fixed the dependencies between CodeGen and MIR by
reorganizing the MIR serialization code - the code that prints out
MIR has been moved to CodeGen, and the MIR library has been renamed
to MIRParser. Now the CodeGen library doesn't depend on the
MIRParser library, thus the circular dependency no longer exists.
--Original Commit Message--
MIR Serialization: print and parse LLVM IR using MIR format.
This commit is the initial commit for the MIR serialization project.
It creates a new library under CodeGen called 'MIR'. This new
library adds a new machine function pass that prints out the LLVM IR
using the MIR format. This pass is then added as a last pass when a
'stop-after' option is used in llc. The new library adds the initial
functionality for parsing of MIR files as well. This commit also
extends the llc tool so that it can recognize and parse MIR input files.
Reviewers: Duncan P. N. Exon Smith, Matthias Braun, Philip Reames
Differential Revision: http://reviews.llvm.org/D9616
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238341
91177308-0d34-0410-b5e6-
96231b3b80d8
Jan Vesely [Wed, 27 May 2015 16:54:10 +0000 (16:54 +0000)]
R600: Rely on TypeLegalizer to use divrem instead of div/rem
reviewer: tstellardAMD
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238337
91177308-0d34-0410-b5e6-
96231b3b80d8
Jan Vesely [Wed, 27 May 2015 16:54:09 +0000 (16:54 +0000)]
SelectionDAG: Don't do libcall on div/rem if divrem is custom
v2: TargetLoweringBase:: -> TargetLowering::
Use Ops array
v3: Explicitly use value 0 for ?DIV
Remove redundant newline
Differential revision: http://reviews.llvm.org/D7803
reviewer: ab
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238336
91177308-0d34-0410-b5e6-
96231b3b80d8
Diego Novillo [Wed, 27 May 2015 16:44:47 +0000 (16:44 +0000)]
Fix PR 23499 - Avoid multiple profile counters for functions in comdat sections.
Counter symbols created for linkonce functions are not discarded by ELF
linkers unless the symbols are placed in the same comdat section as its
associated function.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238335
91177308-0d34-0410-b5e6-
96231b3b80d8
Zoran Jovanovic [Wed, 27 May 2015 15:39:47 +0000 (15:39 +0000)]
[mips][microMIPSr6] Implement SEB and SEH instructions
Differential Revision: http://reviews.llvm.org/D9739
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238333
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Wed, 27 May 2015 15:18:34 +0000 (15:18 +0000)]
Fix NDEBUG build.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238332
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Wed, 27 May 2015 15:14:11 +0000 (15:14 +0000)]
Delete MCSectionData.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238331
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Wed, 27 May 2015 14:55:09 +0000 (14:55 +0000)]
Delete dead code. NFC.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238330
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Wed, 27 May 2015 14:45:54 +0000 (14:45 +0000)]
Stop using MCSectionData in WinCOFFObjectWriter.cpp.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238329
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Wed, 27 May 2015 14:37:12 +0000 (14:37 +0000)]
clang-format WinCOFFObjectWriter.cpp. NFC.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238328
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Wed, 27 May 2015 14:33:39 +0000 (14:33 +0000)]
Remove uses of MCSectionData from MachObjectWriter.cpp.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238327
91177308-0d34-0410-b5e6-
96231b3b80d8
Jozef Kolek [Wed, 27 May 2015 14:19:22 +0000 (14:19 +0000)]
[mips][microMIPSr6] Implement BEQZALC, BGEZALC, BGTZALC, BLEZALC, BLTZALC and BNEZALC instructions
This patch implements microMIPS32r6 BEQZALC, BGEZALC, BGTZALC, BLEZALC, BLTZALC
and BNEZALC instructions using mapping.
Differential Revision: http://reviews.llvm.org/D10031
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238325
91177308-0d34-0410-b5e6-
96231b3b80d8
Elena Demikhovsky [Wed, 27 May 2015 14:09:33 +0000 (14:09 +0000)]
AVX-512: Fixed a bug in extracting subvector from v64i1
By Igor Breger (igor.breger@intel.com)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238322
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Wed, 27 May 2015 13:37:28 +0000 (13:37 +0000)]
Move getSubsectionInsertionPoint to MCSection.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238320
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Wed, 27 May 2015 13:30:50 +0000 (13:30 +0000)]
Remove uses of MCSectionData from ELFObjectWriter. NFC.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238317
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Wed, 27 May 2015 13:05:42 +0000 (13:05 +0000)]
Use operator<< instead of print in a few more places.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238315
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Wed, 27 May 2015 12:13:06 +0000 (12:13 +0000)]
We always have an InstPrinter.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238311
91177308-0d34-0410-b5e6-
96231b3b80d8
Chandler Carruth [Wed, 27 May 2015 09:36:33 +0000 (09:36 +0000)]
Revert r238305: [PM] Use a simpler technique to drop optional analysis
manager arguments...
I have no idea why, but compilers seem to hate this and its late, so I'm
not going to debug it. :: sigh :: This is why we can't have nice things.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238306
91177308-0d34-0410-b5e6-
96231b3b80d8
Chandler Carruth [Wed, 27 May 2015 09:02:51 +0000 (09:02 +0000)]
[PM] Use a simpler technique to drop optional analysis manager arguments
when invoking run methods.
This technique was suggested by Dinesh Dwivedi who also wrote the
original patch. During the code review, they explained to me that this
isn't a fully general technique as we need to know the signatures of the
method candidates. Since this is really a narrower utility, I switched
the names and structure to be more clearly a specialized run method
invoke helper and commented it accordingly. I still think this is
a pretty big win.
Very sorry to Dinesh for the extreme delay in landing this patch. I've
been far to busy poking at other things.
Original review: http://reviews.llvm.org/D3543
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238305
91177308-0d34-0410-b5e6-
96231b3b80d8
Daniel Sanders [Wed, 27 May 2015 08:44:01 +0000 (08:44 +0000)]
Revert r238190 and r238197: [mips] Make TTypeEncoding indirect to allow .eh_frame to be read-only.
This broke the llvm-mips-linux builder and several of our out-of-tree builders.
Initial investigations show that the commit probably isn't the problem but
reverting anyway while I investigate.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238302
91177308-0d34-0410-b5e6-
96231b3b80d8
Elena Demikhovsky [Wed, 27 May 2015 08:15:19 +0000 (08:15 +0000)]
AVX-512: Implemented all forms of sign-extend and zero-extend instructions for KNL and SKX
Implemented DAG lowering for all these forms.
Added tests for DAG lowering and encoding.
By Igor Breger (igor.breger@intel.com)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238301
91177308-0d34-0410-b5e6-
96231b3b80d8
Chandler Carruth [Wed, 27 May 2015 07:20:46 +0000 (07:20 +0000)]
Add some more detailed docs about the current state of Phabricator and
reviwes.llvm.org to help reduce confusion.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238295
91177308-0d34-0410-b5e6-
96231b3b80d8
Quentin Colombet [Wed, 27 May 2015 06:28:41 +0000 (06:28 +0000)]
[X86] Implement the support for shrink-wrapping.
With this patch the x86 backend is now shrink-wrapping capable
and this functionality can be tested by using the
-enable-shrink-wrap switch.
The next step is to make more test and enable shrink-wrapping by
default for x86.
Related to <rdar://problem/
20821487>
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238293
91177308-0d34-0410-b5e6-
96231b3b80d8
Quentin Colombet [Wed, 27 May 2015 06:25:48 +0000 (06:25 +0000)]
[ShrinkWrap] Add a target hook to check whether or not
the target can handle a given basic block as prologue
or epilogue.
Related to <rdar://problem/
20821487>
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238292
91177308-0d34-0410-b5e6-
96231b3b80d8
Matthias Braun [Wed, 27 May 2015 05:12:40 +0000 (05:12 +0000)]
ARMLoadStoreOptimizer: Code cleanup; NFC
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238289
91177308-0d34-0410-b5e6-
96231b3b80d8
Matthias Braun [Wed, 27 May 2015 05:12:39 +0000 (05:12 +0000)]
MachineBasicBlock: Cleanup computeRegisterLiveness()
- Clean documentation comment
- Change the API to accept an iterator so you can actually pass
MachineBasicBlock::end() now.
- Add more "const".
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238288
91177308-0d34-0410-b5e6-
96231b3b80d8
Matthias Braun [Wed, 27 May 2015 05:12:37 +0000 (05:12 +0000)]
MCSymbol: Make print() robust against empty names
This shouldn't happen, but it's nice not to abort when printing broken machine
functions.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238287
91177308-0d34-0410-b5e6-
96231b3b80d8
Rui Ueyama [Wed, 27 May 2015 02:55:04 +0000 (02:55 +0000)]
Object/COFF: Add accessors for import header bitfields.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238277
91177308-0d34-0410-b5e6-
96231b3b80d8
Chandler Carruth [Wed, 27 May 2015 02:49:05 +0000 (02:49 +0000)]
[inliner] Fix the early-exit of the inline cost analysis to correctly
model the dense vector instruction bonuses.
Previously, this code really didn't effectively compute the density of
inlined vector instructions and apply the intended inliner bonus. It
would try to compute it repeatedly while analyzing the function and
didn't handle the case where future vector instructions would tip the
scales back towards the bonus.
Instead, speculatively apply all possible bonuses to the threshold
initially. Once we *know* that a certain bonus can not be applied,
subtract it. This should delay early bailout enough to get much more
consistent results without actually causing us to analyze huge swaths of
code. I expect some (hopefully mild) compile time hit here, and some
swings in performance, but this was definitely the intended behavior of
these bonuses.
This also dramatically simplifies the computation of the bonuses to not
interact with each other in confusing ways. The previous code didn't do
a good job of this and the values for bonuses may be surprising but are
at least now clearly written in the code.
Finally, fix code to be in line with comments and use zero as the
bailout condition.
Patch by Easwaran Raman, with some comment tweaks by me to try and
further clarify what is going on with this code.
http://reviews.llvm.org/D8267
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238276
91177308-0d34-0410-b5e6-
96231b3b80d8
Rui Ueyama [Wed, 27 May 2015 02:40:20 +0000 (02:40 +0000)]
Object/COFF: Add coff_import_header.
This type is described in the PE/COFF spec section 7.1.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238275
91177308-0d34-0410-b5e6-
96231b3b80d8
Filipe Cabecinhas [Wed, 27 May 2015 01:05:40 +0000 (01:05 +0000)]
[BitcodeReader] Change assert to report_fatal_error
It can be triggered by user input.
Bug found with AFL fuzz.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238272
91177308-0d34-0410-b5e6-
96231b3b80d8
Filipe Cabecinhas [Wed, 27 May 2015 00:48:43 +0000 (00:48 +0000)]
[BitstreamReader] Make sure the Array operand type is an encoding
Bug found with AFL fuzz.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238269
91177308-0d34-0410-b5e6-
96231b3b80d8
Filipe Cabecinhas [Wed, 27 May 2015 00:48:37 +0000 (00:48 +0000)]
clang-format a couple of lines
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238268
91177308-0d34-0410-b5e6-
96231b3b80d8
Filipe Cabecinhas [Tue, 26 May 2015 23:52:21 +0000 (23:52 +0000)]
[BitcodeReader] Make sure abbrev records have at least one operand (record code)
Bug found with AFL fuzz.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238265
91177308-0d34-0410-b5e6-
96231b3b80d8
Owen Anderson [Tue, 26 May 2015 23:48:40 +0000 (23:48 +0000)]
Add initial support for the convergent attribute.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238264
91177308-0d34-0410-b5e6-
96231b3b80d8
Filipe Cabecinhas [Tue, 26 May 2015 23:00:56 +0000 (23:00 +0000)]
[BitcodeReader] Sanity check on Comdat ID
Shouldn't be an assert, since user input can trigger it.
Bug found with AFL fuzz.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238261
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Tue, 26 May 2015 21:18:29 +0000 (21:18 +0000)]
clang-format TargetRegistry.h. NFC.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238255
91177308-0d34-0410-b5e6-
96231b3b80d8
Philip Reames [Tue, 26 May 2015 21:16:42 +0000 (21:16 +0000)]
[PlaceSafepoints] Entry safepoint location doesn't need to be a terminator
Long ago, the poll insertion code assumed that the insertion site was a terminator. As a result, the entry selection code would split a basic block to ensure it could pass a terminator. The insertion code was updated quite a while ago - possibly before it ever landed upstream - but the now redundant work was never removed.
While I'm at it, remove a comment which doesn't apply to the upstreamed code.
NFC intended.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238254
91177308-0d34-0410-b5e6-
96231b3b80d8
Philip Reames [Tue, 26 May 2015 21:03:23 +0000 (21:03 +0000)]
[PlaceSafepoints] Cleanup InsertSafepointPoll function
While working on another change, I noticed that the naming in this function was mildly deceptive. While fixing that, I took the oppurtunity to modernize some of the code. NFC intended.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238252
91177308-0d34-0410-b5e6-
96231b3b80d8
Kostya Serebryany [Tue, 26 May 2015 20:57:47 +0000 (20:57 +0000)]
[lib/Fuzzer] make the fuzzing timeout 1200 seconds by default (was: infinity)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238251
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Tue, 26 May 2015 20:37:36 +0000 (20:37 +0000)]
Simplify boolean conditional return statements.
Patch by Richard <legalize@xmission.com>.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238250
91177308-0d34-0410-b5e6-
96231b3b80d8
Rui Ueyama [Tue, 26 May 2015 20:25:30 +0000 (20:25 +0000)]
Object/COFF: Add COFFSymbolRef::isAbsolute().
COFFSymbolRef has many predicates, like isCommon() or
isExternal(), but this predicate was missing.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238247
91177308-0d34-0410-b5e6-
96231b3b80d8
Akira Hatanaka [Tue, 26 May 2015 20:17:20 +0000 (20:17 +0000)]
Remove NoFramePointerElim and NoFramePointerElimOverride from TargetOptions and
remove ExecutionEngine's dependence on CodeGen. NFC.
This is a follow-up to r238080.
Differential Revision: http://reviews.llvm.org/D9830
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238244
91177308-0d34-0410-b5e6-
96231b3b80d8
Adrian Prantl [Tue, 26 May 2015 20:06:51 +0000 (20:06 +0000)]
Use "auto &" in range-based for-loop and remove the extra braces.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238243
91177308-0d34-0410-b5e6-
96231b3b80d8
Adrian Prantl [Tue, 26 May 2015 20:06:48 +0000 (20:06 +0000)]
Fix a use-after-free in a DEBUG output.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238242
91177308-0d34-0410-b5e6-
96231b3b80d8
Kostya Serebryany [Tue, 26 May 2015 19:32:52 +0000 (19:32 +0000)]
[lib/Fuzzer] fix docs
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238236
91177308-0d34-0410-b5e6-
96231b3b80d8
Kostya Serebryany [Tue, 26 May 2015 19:29:33 +0000 (19:29 +0000)]
[lib/Fuzzer] fix build with assertions
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238235
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Tue, 26 May 2015 18:35:10 +0000 (18:35 +0000)]
Print "lock \t foo" instead of "lock \n foo".
This gets gas and llc -filetype=obj to agree on the order of prefixes.
For llvm-mc we need to fix the asm parser to know that it makes a difference
on which line the "lock" is in.
Part of pr23594.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238232
91177308-0d34-0410-b5e6-
96231b3b80d8
Jan Vesely [Tue, 26 May 2015 18:07:22 +0000 (18:07 +0000)]
R600: Use SIGN_EXTEND_INREG for SEXT loads
Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
Reviewed-by: Matt Arsenault <Matthew.Arsenault@amd.com>
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238229
91177308-0d34-0410-b5e6-
96231b3b80d8
Jan Vesely [Tue, 26 May 2015 18:07:21 +0000 (18:07 +0000)]
R600: Add comments to subword private address load lowering code
v2: Use C++ comments and end with periods
Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
Reviewed-by: Matt Arsenault <Matthew.Arsenault@amd.com>
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238228
91177308-0d34-0410-b5e6-
96231b3b80d8
Diego Novillo [Tue, 26 May 2015 17:45:38 +0000 (17:45 +0000)]
Revert "Re-commit changes in r237579 with fix for bug breaking windows builds."
This reverts commit r238201 to fix linking problems in x86 Linux
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-
20150525/278413.html
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238223
91177308-0d34-0410-b5e6-
96231b3b80d8
Matt Arsenault [Tue, 26 May 2015 17:33:15 +0000 (17:33 +0000)]
Make EmitFunctionHeader virtual
This is to fix problems introduced by r232481. For HSAIL,
this function does essentially nothing desirable, and
injects unwanted / incorrect stuff before the function.
The only thing it really needs to do is call EmitFunctionEntryLabel
in this case.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238222
91177308-0d34-0410-b5e6-
96231b3b80d8
Matt Arsenault [Tue, 26 May 2015 17:01:16 +0000 (17:01 +0000)]
Forgot to add lit.local.cfg for new R600 directory
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238218
91177308-0d34-0410-b5e6-
96231b3b80d8
Matt Arsenault [Tue, 26 May 2015 16:59:43 +0000 (16:59 +0000)]
CodeGenPrepare: Don't match addressing modes through addrspacecast
This was resulting in the addrspacecast being removed and incorrectly
replaced with a ptrtoint when sinking.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238217
91177308-0d34-0410-b5e6-
96231b3b80d8
Rui Ueyama [Tue, 26 May 2015 16:24:42 +0000 (16:24 +0000)]
Object/COFF: Define .idata import directory table entry.
This data type is described in the Microsoft PE/COFF spec rev. 8.3 5.4.1.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238214
91177308-0d34-0410-b5e6-
96231b3b80d8
Rui Ueyama [Tue, 26 May 2015 16:20:40 +0000 (16:20 +0000)]
Object: Add Archive::getNumberOfSymbols().
Add a function that returns number of symbols in archive headers.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238213
91177308-0d34-0410-b5e6-
96231b3b80d8
Tom Stellard [Tue, 26 May 2015 15:55:52 +0000 (15:55 +0000)]
R600/SI: Add assembler support for all CI and VI VOP2 instructions
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238211
91177308-0d34-0410-b5e6-
96231b3b80d8
Tom Stellard [Tue, 26 May 2015 15:55:50 +0000 (15:55 +0000)]
AsmMatcherEmitter: Add an option to override custom converters for InstAlias
If there is an InstAlias defined for an instruction that had a custom
converter (AsmMatchConverter), then when the alias is matched,
the custom converter will be used rather than the converter generated
by the InstAlias.
This patch adds the UseInstAsmMatchConverter field to the InstAlias
class, which allows you to override this behavior and force the
converter generated by the InstAlias to be used.
This is required for some future improvemnts to the R600 assembler.
Differential Revision: http://reviews.llvm.org/D9083
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238210
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Tue, 26 May 2015 15:07:25 +0000 (15:07 +0000)]
Replace getOrCreateSectionData with registerSection.
There is now no SectionData to be created.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238208
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Tue, 26 May 2015 14:51:03 +0000 (14:51 +0000)]
Remove dead forward declaration.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238205
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Tue, 26 May 2015 14:48:11 +0000 (14:48 +0000)]
Have getCurrentSectionData return a MCSection.
I will fix the name shortly.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238204
91177308-0d34-0410-b5e6-
96231b3b80d8
Rafael Espindola [Tue, 26 May 2015 14:42:52 +0000 (14:42 +0000)]
Pass a MCSection to getCurrentSectionData.
A step towards merging MCSection and MCSectionData.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238203
91177308-0d34-0410-b5e6-
96231b3b80d8
Luke Cheeseman [Tue, 26 May 2015 13:40:31 +0000 (13:40 +0000)]
Re-commit changes in r237579 with fix for bug breaking windows builds.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238201
91177308-0d34-0410-b5e6-
96231b3b80d8
Luke Cheeseman [Tue, 26 May 2015 13:10:35 +0000 (13:10 +0000)]
Test Commit
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238199
91177308-0d34-0410-b5e6-
96231b3b80d8
Elena Demikhovsky [Tue, 26 May 2015 12:37:17 +0000 (12:37 +0000)]
AVX-512: fixed a bug in arithmetic operations lowering for i1 type
https://llvm.org/bugs/show_bug.cgi?id=23630
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238198
91177308-0d34-0410-b5e6-
96231b3b80d8
Daniel Sanders [Tue, 26 May 2015 12:25:36 +0000 (12:25 +0000)]
Fix warning introduced in r238190 about lack of virtual destructor in MCObjectFileInfo.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238197
91177308-0d34-0410-b5e6-
96231b3b80d8
Elena Demikhovsky [Tue, 26 May 2015 11:32:39 +0000 (11:32 +0000)]
AVX-512: fixed a bug in lowering VSELECT for 512-bit vector
https://llvm.org/bugs/show_bug.cgi?id=23634
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238195
91177308-0d34-0410-b5e6-
96231b3b80d8
Michael Kuperstein [Tue, 26 May 2015 10:47:10 +0000 (10:47 +0000)]
Use std::bitset for SubtargetFeatures.
Previously, subtarget features were a bitfield with the underlying type being uint64_t.
Since several targets (X86 and ARM, in particular) have hit or were very close to hitting this bound, switching the features to use a bitset.
No functional change.
The first several times this was committed (e.g. r229831, r233055), it caused several buildbot failures.
Apparently the reason for most failures was both clang and gcc's inability to deal with large numbers (> 10K) of bitset constructor calls in tablegen-generated initializers of instruction info tables.
This should now be fixed.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238192
91177308-0d34-0410-b5e6-
96231b3b80d8
Daniel Sanders [Tue, 26 May 2015 10:19:18 +0000 (10:19 +0000)]
[mips] Make TTypeEncoding indirect to allow .eh_frame to be read-only.
Summary:
Following on from r209907 which made personality encodings indirect, do the
same for TType encodings. This fixes the case where a try/catch block needs
to generate references to, for example, std::exception in the
.gcc_except_table.
This commit uses DW_EH_PE_sdata8 for N64 as far as is possible at the moment.
However, it is possible to end up with DW_EH_PE_sdata4 when a TargetMachine is
not available. There's no risk of issues with inconsistency here since the
tables are self describing but it does mean there is a small chance of the
PC-relative offset being out of range for particularly large programs.
Reviewers: petarj
Reviewed By: petarj
Subscribers: srhines, joerg, tberghammer, llvm-commits
Differential Revision: http://reviews.llvm.org/D9669
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@238190
91177308-0d34-0410-b5e6-
96231b3b80d8