summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* Add a comment for the test. NFC.Wei Mi2017-09-131-0/+2
| | | | llvm-svn: 313199
* [Transforms] Fix some Clang-tidy modernize-use-using and Include What You ↵Eugene Zelenko2017-09-137-147/+267
| | | | | | Use warnings; other minor fixes (NFC). llvm-svn: 313198
* [RegAlloc] Keep a copy of live interval for the spilled vregs in ↵Wei Mi2017-09-132-24/+553
| | | | | | | | | | | | | HoistSpillHelper. This is to fix PR34502. After rL311401, the live range of spilled vreg will be cleared. HoistSpill need to use the live range of the original vreg before splitting to know the moving range of the spills. The patch saves a copy of live interval for the spilled vreg inside of HoistSpillHelper. Differential Revision: https://reviews.llvm.org/D37578 llvm-svn: 313197
* [Bitcode] Add a compatibility test for 5.0.0 bitcodeVedant Kumar2017-09-132-0/+1705
| | | | llvm-svn: 313196
* Invoke GetInlineCost for legality check before inline functions in ↵Dehao Chen2017-09-133-6/+50
| | | | | | | | | | | | | | | | SampleProfileLoader. Summary: SampleProfileLoader inlines hot functions if it is inlined in the profiled binary. However, the inline needs to be guarded by legality check, otherwise it could lead to correctness issues. Reviewers: eraman, davidxl Reviewed By: eraman Subscribers: sanjoy, llvm-commits Differential Revision: https://reviews.llvm.org/D37779 llvm-svn: 313195
* [CodeGen] Fix some Clang-tidy modernize and Include What You Use warnings; ↵Eugene Zelenko2017-09-1317-177/+317
| | | | | | other minor fixes (NFC). llvm-svn: 313194
* Fix a crash in llvm-nm for a bad Mach-O file that has an N_SECT type symbol ↵Kevin Enderby2017-09-133-0/+14
| | | | | | | | | | | | | | and a zero n_sect value. The code in llvm-nm for Mach-O files to determine the section type for an N_SECT type symbol it will call getSymbolSection() and check for the error, but in the case the n_sect value is zero it will return section_end() (aka nullptr). And the code was using that and crashing instead of just returning a ’s’ for a section or printing (?,?) as it would if getSymbolSection() returned an error. rdar://33136604 llvm-svn: 313193
* Mark static member functions as static in CodeViewDebugAdrian McCarthy2017-09-135-12/+127
| | | | | | | | | | | | | | Summary: To improve CodeView quality for static member functions, we need to make the static explicit. In addition to a small change in LLVM's CodeViewDebug to return the appropriate MethodKind, this requires a small change in Clang to note the staticness in the debug info metadata. Subscribers: aprantl, hiraditya Differential Revision: https://reviews.llvm.org/D37715 llvm-svn: 313192
* Revert "Determine up front which projects are enabled."Zachary Turner2017-09-133-61/+23
| | | | | | | | | | | This was intended to be a generic CMake solution to a problem shared across several projects. It turns out it doesn't interact very well certain CMake configurations, and furthermore the "problem" is actually not a problem, as the problematic code is never executed to begin with. So this really isn't solving anything. llvm-svn: 313191
* Simplify. NFC.Rafael Espindola2017-09-131-1/+1
| | | | | | The isShared case was handled by the isInCurrentDSO check. llvm-svn: 313190
* [ubsan-minimal] Temporarily disable x86_64h testing on DarwinVedant Kumar2017-09-131-0/+5
| | | | | | | | | | | | | | | We're seeing strange issues on the public GreenDragon Darwin bots which we don't understand. x86_64h tests are still being run on pre-Haswell bots despite the added checks in test/ubsan_minimal/lit.common.cfg, which were verified on our internal bots. I'm unable to ssh into the affected public bot, so for now am trying a more aggressive check which disables all x86_64h testing for ubsan-minimal on Darwin. rdar://problem/34409349 llvm-svn: 313189
* Add a helper for checking for weak undef. NFC.Rafael Espindola2017-09-135-9/+21
| | | | llvm-svn: 313188
* Do not use hasArgNoClaim().Rui Ueyama2017-09-132-4/+4
| | | | | | | | | Arg instances can be claimed. After claimed, its `isClaimed` function returns true. We do not use that notion in lld, so using NoClaim versions of functions is just confusing. This patch is to just use hasArg instead of hasArgNoClaim. llvm-svn: 313187
* [OPENMP] Fix types for the target specific parameters in debug mode.Alexey Bataev2017-09-132-30/+37
| | | | | | | Used incorrect types for target specific parameters in debug mode, should use original pointers rather than the pointee types. llvm-svn: 313186
* [Inliner] Add another way to compute full inline cost.Easwaran Raman2017-09-132-5/+8
| | | | | | | | | | | | | | | | Summary: Full inline cost is computed when -inline-cost-full is true or ORE is non-null. This patch adds another way to compute full inline cost by adding a field to InlineParams. This will be used by SampleProfileLoader to check legality of inlining a callee that it wants to inline. Reviewers: danielcdh, haicheng Subscribers: llvm-commits Differential Revision: https://reviews.llvm.org/D37819 llvm-svn: 313185
* Use existing helper. NFC.Rafael Espindola2017-09-131-1/+1
| | | | llvm-svn: 313184
* [unittests] Another speculative fix for changes introduced in rL313156Vedant Kumar2017-09-131-1/+2
| | | | llvm-svn: 313183
* SplitEmptyFunction should be true in the Mozilla coding styleSylvestre Ledru2017-09-131-1/+1
| | | | | | | | | | | | | | | | Summary: As defined here: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style#Classes See for the downstream bug report: https://bugzilla.mozilla.org/show_bug.cgi?id=1399359 Reviewers: Typz, djasper Reviewed By: Typz Subscribers: klimek, cfe-commits Differential Revision: https://reviews.llvm.org/D37795 llvm-svn: 313182
* [unittests] Speculative fix for changes introduced in rL313156Vedant Kumar2017-09-131-1/+3
| | | | llvm-svn: 313181
* Use getUnaliasedOption so that this switch works for option aliases.Rui Ueyama2017-09-131-1/+1
| | | | | | | There are no alises handled by this switch, but getUnaliasesdOption is preferred way of doing this. This is also consistent with ELF and COFF. llvm-svn: 313180
* [docs] Update LeakSanitizer documentation to reflect OS X supportFrancis Ricci2017-09-132-3/+4
| | | | | | | | | | Reviewers: kcc, alekseyshl, kubamracek, glider Subscribers: llvm-commits Differential Revision: https://reviews.llvm.org/D37811 llvm-svn: 313179
* Move a piece of code above some local variable definitions to make their ↵Rui Ueyama2017-09-131-3/+2
| | | | | | scope narrower. llvm-svn: 313178
* [asan] Fix Windows buildVitaly Buka2017-09-131-1/+1
| | | | llvm-svn: 313177
* [LV] Avoid computing the register usage for default VF. NFCAnna Thomas2017-09-132-6/+4
| | | | | | | | | | | These are changes to reduce redundant computations when calculating a feasible vectorization factor: 1. early return when target has no vector registers 2. don't compute register usage for the default VF. Suggested during review for D37702. llvm-svn: 313176
* [MinGW] Only apply -Bstatic to following librariesMartin Storsjo2017-09-132-6/+17
| | | | | | | | | This is how the flag is documented in GNU binutils ld; -Bstatic only applies to -l options after it, until the next -Bdynamic. Differential Revision: https://reviews.llvm.org/D37794 llvm-svn: 313175
* [MinGW] Support dllexport on i386Martin Storsjo2017-09-136-0/+35
| | | | | | | | | | | | In MinGW configurations (GCC, or clang with a *-windows-gnu target), the -export directives in the object file contains the undecorated symbol name, while it is decorated in MSVC configurations. (On the command line, link.exe takes an undecorated symbol name for the -export argument though.) Differential Revision: https://reviews.llvm.org/D37772 llvm-svn: 313174
* [Fuchsia] Set ENABLE_X86_RELAX_RELOCATIONS for Fuchsia buildsPetr Hosek2017-09-131-0/+7
| | | | | | | | | | This is a "Does your linker support it?" option, and all ours do. Patch by Roland McGrath Differential Revision: https://reviews.llvm.org/D37785 llvm-svn: 313173
* Add more tests for OpenCL atomic builtin functionsYaxun Liu2017-09-132-1/+45
| | | | | | | | Add tests for different address spaces and insert some blank lines to make them more readable. Differential Revision: https://reviews.llvm.org/D37742 llvm-svn: 313172
* [AMDGPU] Change addr space of clk_event_t, queue_t and reserve_id_t to globalYaxun Liu2017-09-132-8/+15
| | | | | | Differential Revision: https://reviews.llvm.org/D37703 llvm-svn: 313171
* Fix a misleading phrase in the LangRefSanjoy Das2017-09-131-2/+5
| | | | | | | | | | Reviewers: hfinkel, dberlin Subscribers: mcrosier, llvm-commits Differential Revision: https://reviews.llvm.org/D37432 llvm-svn: 313170
* Driver: Make -fwhole-program-vtables a core option so it can be used from ↵Peter Collingbourne2017-09-132-2/+12
| | | | | | | | | | clang-cl. Also add some missing driver tests for the regular clang driver. Differential Revision: https://reviews.llvm.org/D37787 llvm-svn: 313169
* [compiler-rt] Use SignalContext in ErrorStackOverflow and ErrorDeadlySignalVitaly Buka2017-09-136-74/+78
| | | | | | | | | | | | Summary: Part of https://github.com/google/sanitizers/issues/637 Reviewers: eugenis, alekseyshl, filcab Subscribers: kubamracek, llvm-commits, dberris Differential Revision: https://reviews.llvm.org/D37793 llvm-svn: 313168
* [compiler-rt] Add siginfo into SignalContextVitaly Buka2017-09-138-17/+38
| | | | | | | | | | | | | | | Summary: Information stored there is often been passed along with SignalContext. Part of https://github.com/google/sanitizers/issues/637 Reviewers: eugenis, alekseyshl Subscribers: kubamracek, llvm-commits, dberris Differential Revision: https://reviews.llvm.org/D37792 llvm-svn: 313167
* Refactoring the stride 4 code in the X86interleavedaccess NFCMichael Zuckerman2017-09-131-34/+32
| | | | llvm-svn: 313166
* llvm-dwarfdump: support dumping UUIDs of Mach-O binaries.Adrian Prantl2017-09-1310-14/+464
| | | | | | | This is a feature supported by Darwin dwarfdump. UUIDs are used to associate executables with their .dSYM bundles. llvm-svn: 313165
* [CFG] Fix typo in docblock: blocsk/blocksBrian Gesiak2017-09-131-1/+1
| | | | llvm-svn: 313164
* [unittests] Fix up test after rL313156Vedant Kumar2017-09-131-2/+3
| | | | | Bot: http://green.lab.llvm.org/green/job/clang-stage1-cmake-RA-incremental/42421 llvm-svn: 313163
* Attempt to fix MSVC build.Alexander Kornienko2017-09-131-3/+4
| | | | llvm-svn: 313162
* [InstSimplify] regenerate checks; NFCSanjay Patel2017-09-131-15/+26
| | | | llvm-svn: 313161
* [mips] Add unitests to check parsing MIPS triples. NFCSimon Atanasyan2017-09-131-0/+30
| | | | llvm-svn: 313160
* Add options to dump PGO counts in text.Hiroshi Yamauchi2017-09-133-34/+57
| | | | | | | | | | | | | | | | | Summary: Added text options to -pgo-view-counts and -pgo-view-raw-counts that dump block frequency and branch probability info in text. This is useful when the graph is very large and complex (the dot command crashes, lines/edges too close to tell apart, hard to navigate without textual search) or simply when text is preferred. Reviewers: davidxl Reviewed By: davidxl Subscribers: llvm-commits Differential Revision: https://reviews.llvm.org/D37776 llvm-svn: 313159
* [ThinLTO] AliasSummary should not have any referencesTeresa Johnson2017-09-133-7/+9
| | | | | | | | | | | | Summary: References should only be on the aliasee. Reviewers: pcc Subscribers: llvm-commits, inglorion Differential Revision: https://reviews.llvm.org/D37814 llvm-svn: 313158
* ThinLTO: Correctly follow aliasee references when dead stripping.Peter Collingbourne2017-09-133-8/+29
| | | | | | | | | | | | | | | | | | We were previously handling aliases during dead stripping by adding the aliased global's "original name" GUID to the worklist. This will lead to incorrect behaviour if the global has local linkage because the original name GUID will not correspond to the global's GUID in the summary. Because an alias is just another name for the global that it references, there is no need to mark the referenced global as used, or to follow references from any other copies of the global. So all we need to do is to follow references from the aliasee's summary instead of the alias. Differential Revision: https://reviews.llvm.org/D37789 llvm-svn: 313157
* Update users of llvm::sys::ExecuteAndWait etc.Alexander Kornienko2017-09-135-34/+20
| | | | | | | | | | | | Summary: Clang part of https://reviews.llvm.org/D37563 Reviewers: bkramer Subscribers: vsk, cfe-commits Differential Revision: https://reviews.llvm.org/D37564 llvm-svn: 313156
* Convenience/safety fix for llvm::sys::Execute(And|No)WaitAlexander Kornienko2017-09-1312-62/+60
| | | | | | | | | | | | | | | | | | | | Summary: Change the type of the Redirects parameter of llvm::sys::ExecuteAndWait, ExecuteNoWait and other APIs that wrap them from `const StringRef **` to `ArrayRef<Optional<StringRef>>`, which is safer and simplifies the use of these APIs (no more local StringRef variables just to get a pointer to). Corresponding clang changes will be posted as a separate patch. Reviewers: bkramer Reviewed By: bkramer Subscribers: vsk, llvm-commits Differential Revision: https://reviews.llvm.org/D37563 llvm-svn: 313155
* Remove CopyRelSecOff from SharedSymbol.Rafael Espindola2017-09-133-7/+8
| | | | | | | This reduces the size of SharedSymbol which in turn reduces the size of Symbol from 88 to 80 bytes. llvm-svn: 313154
* Fix bot failures by requiring x86 target in new testTeresa Johnson2017-09-131-0/+2
| | | | | | | | The test added in r313151 requires a target triple since it is running through code generation. Fix bot failures by requiring an x86 target. llvm-svn: 313153
* This adds the _Float16 preprocessor macro definitions.Sjoerd Meijer2017-09-134-16/+112
| | | | | | Differential Revision: https://reviews.llvm.org/D34695 llvm-svn: 313152
* [ThinLTO] For SamplePGO, need to handle ICP targets consistently in thin linkTeresa Johnson2017-09-134-11/+154
| | | | | | | | | | | | | | | | | | | | | | Summary: SamplePGO indirect call profiles record the target as the original GUID for statics. The importer had special handling to map to the normal GUID in that case. The dead global analysis needs the same treatment or inconsistencies arise, resulting in linker unsats due to some dead symbols being exported and kept, leaving in references to other dead symbols that are removed. This can happen when a SamplePGO profile collected by one binary is used for a different binary, so the indirect call profiles may not accurately reflect live targets. Reviewers: danielcdh Subscribers: mehdi_amini, inglorion, llvm-commits, eraman Differential Revision: https://reviews.llvm.org/D37783 llvm-svn: 313151
* [clang-tidy] fixed misc-unused-parameters omitting parameters default valueAlexander Kornienko2017-09-132-4/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: Bug: https://bugs.llvm.org/show_bug.cgi?id=34450 **Problem:** Clang-tidy check misc-unused-parameters omits parameter default value what results in its complete removal. Compilation errors might occur after clang-tidy fix. **Patch description:** Changed removal range. The range should end after parameter declarator, not after whole parameter declaration (which might contain a default value). Reviewers: alexfh, xazax.hun Reviewed By: alexfh Subscribers: JDevlieghere, cfe-commits Tags: #clang-tools-extra Patch by Pawel Maciocha! Differential Revision: https://reviews.llvm.org/D37566 llvm-svn: 313150
OpenPOWER on IntegriCloud