summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* [IRTranslator] Translate unconditional branches.Quentin Colombet2016-03-115-1/+59
| | | | llvm-svn: 263265
* [MachineIRBuilder] Rework buildInstr API to maximize code reuse.Quentin Colombet2016-03-112-24/+34
| | | | llvm-svn: 263264
* [IRTranslator] Update getOrCreateVReg API to use references.Quentin Colombet2016-03-112-12/+13
| | | | | | | A value that we want to keep in a virtual register cannot be null. Reflect that in the API. llvm-svn: 263263
* [MachineIRBuilder] Rename the setter of MF for consistency with the getter.Quentin Colombet2016-03-113-3/+3
| | | | llvm-svn: 263262
* [MachineIRBuilder] Rename the setter for MBB for consistency with the getter.Quentin Colombet2016-03-113-5/+7
| | | | llvm-svn: 263261
* [IRTranslator] Update getOrCreateBB API to use references.Quentin Colombet2016-03-112-5/+7
| | | | | | A null basic block is invalid, so just pass a reference. llvm-svn: 263260
* [GlobalISel][Target] Add an opcode for unconditional branch.Quentin Colombet2016-03-115-7/+21
| | | | llvm-svn: 263259
* Remove PreserveNames template parameter from IRBuilderMehdi Amini2016-03-1114-41/+32
| | | | | | | | | | | | | | | Summary: Following r263086, we are now relying on a flag on the Context to discard Value names in release builds. Reviewers: chandlerc Subscribers: mzolotukhin, llvm-commits Differential Revision: http://reviews.llvm.org/D18023 From: Mehdi Amini <mehdi.amini@apple.com> llvm-svn: 263258
* Remove compile time PreserveName switch based on NDEBUGMehdi Amini2016-03-115-36/+15
| | | | | | | | | | | | | | | Summary: Following r263086, we are now relying on a flag on the Context to discard Value names in release builds. Reviewers: chandlerc Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D18024 From: Mehdi Amini <mehdi.amini@apple.com> llvm-svn: 263257
* Do not specialize IRBuilder to strip names in SROAMehdi Amini2016-03-111-22/+10
| | | | | | | | | | | | | | | | | Summary: Following r263086, we are replacing this by a runtime check. More cleanup will follow on the IRBuilder itself, but I submitted this patch separately as SROA has a fancy "prefixInserter" class that needs extra-love. Reviewers: chandlerc Subscribers: llvm-commits Differential Revision: http://reviews.llvm.org/D18022 From: Mehdi Amini <mehdi.amini@apple.com> llvm-svn: 263256
* [misched] Fix a truncation issue from r263021.Chad Rosier2016-03-111-1/+1
| | | | | | | | | The truncation was causing the sorting algorithm to behave oddly when comparing positive and negative offsets. Fortunately, this doesn't currently happen in practice and was exposed by a WIP. Thus, I can't test this change now, but the follow on patch will. llvm-svn: 263255
* Yet more MSVC fixes.Rafael Espindola2016-03-111-3/+6
| | | | llvm-svn: 263253
* [ELF] - Change all messages to lowercase to be consistent.George Rimar2016-03-1125-156/+141
| | | | | | | | | | | | | | That is directly opposite to http://reviews.llvm.org/D18045, which was reverted. This patch changes all messages to start from lowercase letter if they were not before. That is done to be consistent with clang. Differential revision: http://reviews.llvm.org/D18085 llvm-svn: 263252
* More MSVC fixes.Rafael Espindola2016-03-113-4/+5
| | | | llvm-svn: 263251
* [PM] Sink the "Expression" type for GVN into the class as a privateChandler Carruth2016-03-112-18/+20
| | | | | | | | | | | | | | | member type. Because of how this type is used by the ValueTable, it cannot actually have hidden visibility. GCC actually nicely warns about this but Clang just silently ... I don't even know. =/ We should do a better job either way though. This should resolve a bunch of the GCC warnings about visibility that the port of GVN triggered and make the visibility story a bit more correct. llvm-svn: 263250
* Trying to fix the MSVC build.Rafael Espindola2016-03-111-5/+6
| | | | llvm-svn: 263249
* Avoid calling getNamedValue.Rafael Espindola2016-03-113-61/+76
| | | | | | | In lld we usually avoid hash lookups. In addition to that, IR names are not fully mangled, so it is best to avoid using them whenever possible. llvm-svn: 263248
* More UTF string conversion wrappersMarianne Mailhot-Sarrasin2016-03-114-3/+162
| | | | | | | | | | | Added new string conversion wrappers that convert between `std::string` (of UTF-8 bytes) and `std::wstring`, which is particularly useful for Win32 interop. Also fixed a missing string conversion for `getenv` on Win32, using these new wrappers. The motivation behind this is to provide the support functions required for LLDB to work properly on Windows with non-ASCII data; however, the functions are not LLDB specific. Patch by cameron314 Differential Revision: http://reviews.llvm.org/D17549 llvm-svn: 263247
* Revert r263036, it's ABI-breaking.Nico Weber2016-03-113-46/+5
| | | | llvm-svn: 263246
* Add tests for ARM Cortex-R8Alexandros Lamprineas2016-03-111-0/+3
| | | | | | | | | | | Add command-line tests for ARM Cortex-R8 checking that the driver calls clang -cc1 with the correct little-endian/big-endian, and ARM/Thumb triple. Patch by Pablo Barrio <pablo.barrio@arm.com> Differential Revision: http://reviews.llvm.org/D18052 llvm-svn: 263245
* Don't invent names for STT_SECTION symbols.Rafael Espindola2016-03-112-7/+2
| | | | | | We should just copy the symbol names with -r. llvm-svn: 263244
* Simplify test.Rafael Espindola2016-03-111-8/+19
| | | | | | We only need one run of llmv-readobj. llvm-svn: 263243
* [AMDGPU] Fix VOPC instruction operand namingsValery Pykhtin2016-03-112-2/+27
| | | | | | Differential Revision: http://reviews.llvm.org/D17966 llvm-svn: 263242
* Simplify test. NFC.Rafael Espindola2016-03-111-2/+2
| | | | llvm-svn: 263241
* This reverts the r263125George Rimar2016-03-1121-74/+74
| | | | | | | | | | | | | | | | | | | | It was discussed to make all messages be lowercase to be consistent with clang. (also reverts the r263128 which fixed build bot fail after r263125) Original commit message: [ELF] - Consistent spelling for error/warning messages Previously error and warnings were not consistent in lld. Some of them started from lowercase letter, others from uppercase. Also there was one or two which had a dot at the end. This patch changes all messages to start from uppercase letter if they were not before. Differential revision: http://reviews.llvm.org/D18045 llvm-svn: 263240
* [X86][AVX] Fixed issue where a long chain of shuffles could attempt to ↵Simon Pilgrim2016-03-112-9/+50
| | | | | | | | combine to a single (illegal) PSHUFB instruction. Its not enough that we test for SSSE3 - that's only OK for 128-bit vectors - we also need to test for AVX2 / AVX512BW for 256/512 bit vector cases. llvm-svn: 263239
* Run dos2unix. NFC.Rafael Espindola2016-03-115-141/+141
| | | | llvm-svn: 263238
* Represent local symbols with DefinedRegular.Rafael Espindola2016-03-116-70/+41
| | | | llvm-svn: 263237
* [ELF][MIPS] Update comment about creation local GOT entries for non-local ↵Simon Atanasyan2016-03-111-3/+5
| | | | | | symbols. NFC. llvm-svn: 263236
* [AA] Make BasicAA just require domtree.Chandler Carruth2016-03-111-4/+5
| | | | | | | | | | | | | This doesn't change how many times we construct domtrees in the normal pipeline, and it removes fragility and instability where basic-aa may not be run in time to see domtrees because they happen to be constructed afterward. This isn't quite as clean as the change to memdep because there is a mode where basic-aa specifically runs without domtrees -- in the hacking version used by function-attrs with the legacy pass manager. llvm-svn: 263234
* Fixed MemoryCache L1 cache flushMarianne Mailhot-Sarrasin2016-03-114-1/+86
| | | | | | | | Use the same method to find the cache line as in Read(). Differential Revision: http://reviews.llvm.org/D18050 llvm-svn: 263233
* [memdep] Just require domtree for memdep.Chandler Carruth2016-03-112-18/+13
| | | | | | | | | | | | | | This doesn't cause us to construct dominator trees any more often in the normal pipeline, and removes an entire mode of memdep that needed to be reasoned about and maintained. Perhaps more importantly, it removes the ability for the results of memdep to be different because of accidental pass scheduling goofs or the order of evaluation of 'getResult' calls. Essentially, 'getCachedResult', unless across IR-unit boundaries, is extremely dangerous. We need to work much harder to avoid it (or its analog in the old pass manager). llvm-svn: 263232
* [PM] The order of evaluation of these analyses is actually significant,Chandler Carruth2016-03-111-5/+10
| | | | | | | | | | | | | | | | | | | | much to my horror, so use variables to fix it in place. This terrifies me. Both basic-aa and memdep will provide more precise information when the domtree and/or the loop info is available. Because of this, if your pass (like GVN) requires domtree, and then queries memdep or basic-aa, it will get more precise results. If it does this in the other order, it gets less precise results. All of the ideas I have for fixing this are, essentially, terrible. Here I've just caused us to stop having unspecified behavior as different implementations evaluate the order of these arguments differently. I'm actually rather glad that they do, or the fragility of memdep and basic-aa would have gone on unnoticed. I've left comments so we don't immediately break this again. This should fix bots whose host compilers evaluate the order of arguments differently from Clang. llvm-svn: 263231
* Remove a redundant cast.Rafael Espindola2016-03-111-2/+2
| | | | llvm-svn: 263230
* [mips] MIPSR6 Instruction itinerariesVasileios Kalintiris2016-03-115-95/+216
| | | | | | | | | | | | | | Summary: Defines instruction itineraries for common MIPSR6 instructions. Patch by Simon Dardis. Reviewers: vkalintiris Subscribers: MatzeB, dsanders, llvm-commits Differential Revision: http://reviews.llvm.org/D17198 llvm-svn: 263229
* Simplify now that local symbols can use getVA.Rafael Espindola2016-03-111-50/+42
| | | | | | | | It is really odd that Mips differentiates symbols that are born local and those that become local because of hidden visibility. I don't know enough mips to known if this is a bug or not. llvm-svn: 263228
* [ELF] - Early continue in InputSectionBase<ELFT>::relocate(). NFC.George Rimar2016-03-111-3/+3
| | | | llvm-svn: 263227
* [ELF] - Evaluate addend earlier and use it instead getAddend() calls. NFC.George Rimar2016-03-111-6/+3
| | | | llvm-svn: 263226
* Compute value of local symbol with getVA.Rafael Espindola2016-03-114-93/+80
| | | | llvm-svn: 263225
* Remember the input section of locals.Rafael Espindola2016-03-115-31/+21
| | | | | | This is already a simplification, but will allow much more. llvm-svn: 263224
* [PM] Update Kaleidoscope with the new header file.Chandler Carruth2016-03-114-0/+4
| | | | llvm-svn: 263223
* Create a SymbolBody for locals.Rafael Espindola2016-03-1113-198/+183
| | | | | | pr26878 shows a case where locals have to be in the got. llvm-svn: 263222
* [clang-tidy] Make 'modernize-use-nullptr' check ignores NULL marcos used in ↵Haojian Wu2016-03-112-0/+16
| | | | | | | | | | | | other macros. Reviewers: bkramer, alexfh Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D17958 llvm-svn: 263221
* [mips] Range check simm4.Daniel Sanders2016-03-118-37/+56
| | | | | | | | | | | | Summary: Reviewers: vkalintiris Subscribers: dsanders, llvm-commits Differential Revision: http://reviews.llvm.org/D16811 llvm-svn: 263220
* [PM] Make the AnalysisManager parameter to run methods a reference.Chandler Carruth2016-03-1159-199/+186
| | | | | | | | | | | | This was originally a pointer to support pass managers which didn't use AnalysisManagers. However, that doesn't realistically come up much and the complexity of supporting it doesn't really make sense. In fact, *many* parts of the pass manager were just assuming the pointer was never null already. This at least makes it much more explicit and clear. llvm-svn: 263219
* [Sanitizer][MIPS] internal lstat and fstat for mips64Mohit K. Bhakkad2016-03-111-0/+16
| | | | | | | | | | Reviewers: dvyukov Subscribers: jaydeep, sagar, llvm-commits Differential Revision: http://reviews.llvm.org/D18078 llvm-svn: 263218
* [PM] Rename the CRTP mixin base classes for the new pass manager toChandler Carruth2016-03-1138-99/+113
| | | | | | | | | | | | | | | | | | | | clarify their purpose. Firstly, call them "...Mixin" types so it is clear that there is no type hierarchy being formed here. Secondly, use the term 'Info' to clarify that they aren't adding any interesting *semantics* to the passes or analyses, just exposing APIs used by the management layer to get information about the pass or analysis. Thanks to Manuel for helping pin down the naming confusion here and come up with effective names to address it. In case you already have some out-of-tree stuff, the following should be roughly what you want to update: perl -pi -e 's/\b(Pass|Analysis)Base\b/\1InfoMixin/g' llvm-svn: 263217
* [PM] Implement the final conclusion as to how the analysis IDs shouldChandler Carruth2016-03-1147-81/+182
| | | | | | | | | | | | | | | | | | | | work in the face of the limitations of DLLs and templated static variables. This requires passes that use the AnalysisBase mixin provide a static variable themselves. So as to keep their APIs clean, I've made these private and befriended the CRTP base class (which is the common practice). I've added documentation to AnalysisBase for why this is necessary and at what point we can go back to the much simpler system. This is clearly a better pattern than the extern template as it caught *numerous* places where the template magic hadn't been applied and things were "just working" but would eventually have broken mysteriously. llvm-svn: 263216
* [InstCombine] Use Twines to generate names.Benjamin Kramer2016-03-111-15/+5
| | | | | | | | | Since the names are used in a loop this does more work in debug builds. In release builds value names are generally discarded so we don't have to do the concatenation at all. It's also simpler code, no functional change intended. llvm-svn: 263215
* [ELF] - Move initSymbols() to Driver.cpp. NFC.George Rimar2016-03-113-16/+8
| | | | | | | That is followup for http://reviews.llvm.org/D18047 patch. initSymbols() moved to Driver.cpp and made static. llvm-svn: 263214
OpenPOWER on IntegriCloud