- Sep 26, 2012
-
-
Nico Weber authored
Found with find test -type f | xargs grep RUN: | grep '%clang' | grep -iv '%s' | grep -v '%t' | grep -v '\\$' llvm-svn: 164678
-
Nico Weber authored
llvm-svn: 164677
-
Sylvestre Ledru authored
For example, under a Linux chroot, /proc/ might not be mounted. Therefor, we test if this file exist. If it is the case, use it (the current behavior). Otherwise, we fall back to the detection used by *BSD. The issue has been reported initially on the Debian bug tracker: http://bugs.debian.org/674588 llvm-svn: 164676
-
Michael Liao authored
llvm-svn: 164675
-
Michael Liao authored
llvm-svn: 164674
-
Michael Liao authored
llvm-svn: 164673
-
Nico Weber authored
Add a test for PR13924. Do not revert the test added in r163022, it surprisingly still passes even after reverting the code changes. llvm-svn: 164672
-
Duncan Sands authored
llvm-svn: 164671
-
Craig Topper authored
llvm-svn: 164670
-
Chandler Carruth authored
This should really, really fix PR13916. For real this time. The underlying bug is... a bit more subtle than I had imagined. The setup is a code pattern that leads to an @llvm.memcpy call with two equal pointers to an alloca in the source and dest. Now, not any pattern will do. The alloca needs to be formed just so, and both pointers should be wrapped in different bitcasts etc. When this precise pattern hits, a funny sequence of events transpires. First, we correctly detect the potential for overlap, and correctly optimize the memcpy. The first time. However, we do simplify the set of users of the alloca, and that causes us to run the alloca back through the SROA pass in case there are knock-on simplifications. At this point, a curious thing has happened. If we happen to have an i8 alloca, we have direct i8 pointer values. So we don't bother creating a cast, we rewrite the arguments to the memcpy to dircetly refer to the alloca. Now, in an unrelated area of the pass, we have clever logic which ensures that when visiting each User of a particular pointer derived from an alloca, we only visit that User once, and directly inspect all of its operands which refer to that particular pointer value. However, the mechanism used to detect memcpy's with the potential to overlap relied upon getting visited once per *Use*, not once per *User*. This is always true *unless* the same exact value is both source and dest. It turns out that almost nothing actually produces that pattern though. We can hand craft test cases that more directly test this behavior of course, and those are included. Also, note that there is a significant missed optimization here -- we prove in many cases that there is a non-volatile memcpy call with identical source and dest addresses. We shouldn't prevent splitting the alloca in that case, and in fact we should just remove such memcpy calls eagerly. I'll address that in a subsequent commit. llvm-svn: 164669
-
Craig Topper authored
llvm-svn: 164668
-
Craig Topper authored
Replace calls to getSizeInBits with getExtendedSizeInBits since its already known its an extended type. llvm-svn: 164667
-
Craig Topper authored
llvm-svn: 164666
-
Logan Chien authored
In the C programming language, we have to add the "struct" keyword. Otherwise, the compiler will emit error message. llvm-svn: 164665
-
Craig Topper authored
llvm-svn: 164664
-
Craig Topper authored
llvm-svn: 164663
-
Bill Wendling authored
scalar-to-vector conversion that we cannot handle. For instance, when an invalid constraint is used in an inline asm statement. <rdar://problem/12284092> llvm-svn: 164662
-
Ted Kremenek authored
llvm-svn: 164661
-
Nico Weber authored
llvm-svn: 164660
-
Michael Liao authored
- Instead of embedding 'lock' into each mnemonic of atomic instructions except 'xchg', we teach X86 assembly printer to output 'lock' prefix similar to or consistent with code emitter. llvm-svn: 164659
-
Eli Friedman authored
typeid (and a couple other non-standard places where we can transform an unevaluated expression into an evaluated expression) is special because it introduces an an expression evaluation context, which conflicts with the mechanism to compute the current lambda mangling context. PR12123. I would appreciate if someone would double-check that we get the mangling correct with this patch. llvm-svn: 164658
-
Bill Wendling authored
scalar-to-vector conversion that we cannot handle. For instance, when an invalid constraint is used in an inline asm statement. <rdar://problem/12284092> llvm-svn: 164657
-
Eli Friedman authored
enough information so we can mangle them correctly in cases involving dependent parameter types. (This specifically impacts cases involving null pointers and cases involving parameters of reference type.) Fix the mangler to use this information instead of trying to scavenge it out of the parameter declaration. <rdar://problem/12296776>. llvm-svn: 164656
-
Richard Smith authored
for a type alias template can appear as sugar at any level of desugaring, just like a TypedefType. llvm-svn: 164655
-
Michael Ilseman authored
Fixed issue with Release build. llvm-svn: 164654
-
Sean Callanan authored
a crash if the path to be remaped was NULL. <rdar://problem/12371888> llvm-svn: 164653
-
Jordan Rose authored
The Apple buildbots are set up to pass --target to configure for both cross- and non-cross-compile builds, and the standard autoconf response to this is to set the program prefix to '<target>-'. Until we can figure out the proper way to handle this (don't pass --target? pass an explicit --program-prefix=""? don't auto-populate program_prefix with target_alias?) it's more important to keep the buildbots running. This reverts r164633 / ba48ceb1a3802e20e781ef04ea2573ffae2ac414. llvm-svn: 164651
-
Ted Kremenek authored
llvm-svn: 164649
-
Jim Ingham authored
llvm-svn: 164648
-
Jason Molenda authored
is_host == true so PlatformDarwin knows it can do same-host operations like process lookup. <rdar://problem/12296249> llvm-svn: 164647
-
Michael J. Spencer authored
llvm-svn: 164644
-
Ted Kremenek authored
llvm-svn: 164643
-
Akira Hatanaka authored
llvm-svn: 164642
-
Nick Lewycky authored
only a missed optimization opportunity if the store is over-aligned, but a miscompile if the store's new type has a higher natural alignment than the memcpy did. Fixes PR13920! llvm-svn: 164641
-
Reed Kotler authored
llvm-svn: 164640
-
Jordan Rose authored
This makes the behavior clearer concerning literals with the maximum number of digits. For a 32-bit example, 4,000,000,000 is a valid uint32_t, but 5,000,000,000 is not, so we'd have to count 10-digit decimal numbers as "unsafe" (meaning we have to check for overflow when parsing them, just as we would for numbers with 11 digits or higher). This is the same, only with 64 bits to play with. No functionality change. llvm-svn: 164639
-
Chad Rosier authored
rdar://12299433 llvm-svn: 164638
-
- Sep 25, 2012
-
-
Nick Lewycky authored
reason we were getting two of the same alloca is because of a memmove/memcpy which had the same alloca in both the src and dest. Now we detect that case directly. This has the same testcase as before, but fixes a clang test CodeGenObjC/exceptions.m which runs clang -O2. llvm-svn: 164636
-
Nick Lewycky authored
Chandler, it's not obvious that it's okay that this alloca gets into the list twice to begin with. Please review and see whether this is the fix you really want, but I wanted to get a fix checked in quickly. llvm-svn: 164634
-
Sebastian Pop authored
llvm-svn: 164633
-