| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
std::vector reference. NFC
llvm-svn: 241430
|
|
|
|
|
|
| |
Record's DefInit. I broke this when I fixed memory leaks recently. Remove the DenseMap that mapped Record's to DefInit.
llvm-svn: 240524
|
|
|
|
|
|
| |
Apparently, the style needs to be agreed upon first.
llvm-svn: 240390
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The patch is generated using this command:
tools/clang/tools/extra/clang-tidy/tool/run-clang-tidy.py -fix \
-checks=-*,llvm-namespace-comment -header-filter='llvm/.*|clang/.*' \
llvm/lib/
Thanks to Eugene Kosov for the original patch!
llvm-svn: 240137
|
|
|
|
|
|
| |
integers. NFC
llvm-svn: 239210
|
|
|
|
|
|
| |
other formatting fixes. NFC
llvm-svn: 239209
|
|
|
|
| |
llvm-svn: 239208
|
|
|
|
|
|
| |
couple short lived variables. NFC
llvm-svn: 239207
|
|
|
|
|
|
| |
inner 'if's. NFC
llvm-svn: 239206
|
|
|
|
| |
llvm-svn: 239205
|
|
|
|
| |
llvm-svn: 239022
|
|
|
|
|
|
| |
Name to reduce memory usage.
llvm-svn: 239021
|
|
|
|
| |
llvm-svn: 238808
|
|
|
|
| |
llvm-svn: 238806
|
|
|
|
| |
llvm-svn: 238805
|
|
|
|
|
|
| |
be removed. NFC
llvm-svn: 238727
|
|
|
|
|
|
| |
typeIsConvertibleTo was just calling baseClassOf(this) on the argument passed to it, but there weren't different signatures for baseClassOf so passing 'this' didn't really do anything interesting. typeIsConvertibleTo could have just been a non-virtual method in RecTy. But since that would be kind of a silly method, I instead re-distributed the logic from baseClassOf into typeIsConvertibleTo.
llvm-svn: 238648
|
|
|
|
|
|
| |
the conversions in convertInitializerTo directly. This saves a bunch of vtable entries. NFC
llvm-svn: 238646
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If the type isn't trivially moveable emplace can skip a potentially
expensive move. It also saves a couple of characters.
Call sites were found with the ASTMatcher + some semi-automated cleanup.
memberCallExpr(
argumentCountIs(1), callee(methodDecl(hasName("push_back"))),
on(hasType(recordDecl(has(namedDecl(hasName("emplace_back")))))),
hasArgument(0, bindTemporaryExpr(
hasType(recordDecl(hasNonTrivialDestructor())),
has(constructExpr()))),
unless(isInTemplateInstantiation()))
No functional change intended.
llvm-svn: 238602
|
|
|
|
|
|
|
|
| |
TernOpInit as they weren't able to be called.
I don't think converting the inputs to the Ops was the right behavior anyway.
llvm-svn: 238543
|
|
|
|
|
|
| |
Creating temporary std::strings there is unnecessary.
llvm-svn: 238412
|
|
|
|
| |
llvm-svn: 238399
|
|
|
|
| |
llvm-svn: 238398
|
|
|
|
|
|
|
|
| |
that makes more sense (at least to me).
The old code had a bug if the description was between 75 and 85 characters or so as it substracted PSLen from Desc.size() instead of MAX_LINE_LEN in the compare. It also calculated odd values for PosE on the last split and just let StringRef::slice take care of it being larger than the description string.
llvm-svn: 238187
|
|
|
|
|
|
|
|
| |
ensure the result is a positive number.
I think the fact that it was explicitly excluding 0 kept this from being a tautology. The exclusion of 0 for the old math was also a bug that's easily hit if the description gets split into multiple lines.
llvm-svn: 238186
|
|
|
|
|
|
| |
tablegen header. Minor cleanup in surrounding code.
llvm-svn: 238185
|
|
|
|
| |
llvm-svn: 238181
|
|
|
|
| |
llvm-svn: 238180
|
|
|
|
|
|
| |
the definition since it's already declared in a header file.
llvm-svn: 238178
|
|
|
|
| |
llvm-svn: 238177
|
|
|
|
|
|
|
|
|
| |
We had not been trying hard enough to resolve def names inside multiclasses
that had complex concatenations, etc. Now we'll try harder.
Patch by Amaury Sechet!
llvm-svn: 237877
|
|
|
|
|
|
| |
just generally consistent across all of the overloads.
llvm-svn: 237775
|
|
|
|
| |
llvm-svn: 237774
|
|
|
|
|
|
| |
These were the old names for these operations long ago. NFC
llvm-svn: 237514
|
|
|
|
|
|
| |
some unnecessary curly braces from the same area.
llvm-svn: 237513
|
|
|
|
|
|
| |
portion of the body based on what will also be the terminating condition. NFC
llvm-svn: 237511
|
|
|
|
|
|
|
|
| |
Found by ubsan. This was taking a bool and left shifting by 32 - the
result is 64 bit, so we should really do the math in a type it fits
in.
llvm-svn: 237345
|
|
|
|
|
|
|
|
| |
functional change intended.
The outer if had 3 separate conditions ORed together and then the inner ifs detected which of the three conditions it was by using only a portion of the specific condition. Just put the whole condition in each inner if and remove the outer if.
llvm-svn: 237343
|
|
|
|
| |
llvm-svn: 237342
|
|
|
|
|
|
| |
if it was just comparing to 0. NFC.
llvm-svn: 237340
|
|
|
|
|
|
| |
ArrayRef iterators. NFC
llvm-svn: 237231
|
|
|
|
| |
llvm-svn: 237090
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
cleanups.
Also, change code in tablegen which printed a message and then called
"exit(1)" to use PrintFatalError, instead.
This fixes instances where an empty output file was left behind after
a failed tablegen invocation, which would confuse subsequent ninja
runs into not attempting to rebuild.
Differential Revision: http://reviews.llvm.org/D9608
llvm-svn: 237058
|
|
|
|
| |
llvm-svn: 236398
|
|
|
|
|
|
| |
unnecessary curly braces. NFC
llvm-svn: 236397
|
|
|
|
|
|
| |
end of previous line. NFC
llvm-svn: 236206
|
|
|
|
| |
llvm-svn: 236205
|
|
|
|
|
|
| |
Fold an assignment into an if. Use auto on the result of a couple dyn_casts. NFC
llvm-svn: 236204
|
|
|
|
| |
llvm-svn: 236089
|
|
|
|
| |
llvm-svn: 236088
|