Author: ctopper
Date: Sun Nov 29 00:50:33 2015
New Revision: 254247
URL: http://llvm.org/viewvc/llvm-project?rev=254247&view=rev
Log:
[X86] Remove temporary variables from intrinsic macros. NFC
Modified:
cfe/trunk/lib/Headers/avx512fintrin.h
Modified: cfe/trunk/lib/Headers/avx512fintrin.h
UR
majnemer added a subscriber: majnemer.
majnemer added a comment.
Is it possible to have a test which covers this change?
Comment at: lib/CodeGen/CGExprScalar.cpp:2568
@@ -2567,1 +2567,3 @@
+ // that does not have any other uses.
if (llvm::BinaryOperator* LHSBinOp = dyn_cast(
dcoughlin added a comment.
Gabor, thanks for taking a look at this!
I'm not sure `RegionStore::getBinding()` is the right place to put this logic.
First, not all dereferences of a `std::nullptr_t` value go through
`getBinding()`, so, for example, the following snippet triggers the assertion
ev
scanon added a comment.
Friendly ping.
http://reviews.llvm.org/D14891
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Hi,
Once, long ago, I started working on this checker callback, but forgot
about it. I have decided to finish it now. Original discussion:
http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20131216/095565.html
The motivation was (pipermail doesn't seem to have my original mail, for
some rea
berenm added a comment.
I don't have commit access, if you don't mind landing it for me :)
http://reviews.llvm.org/D14325
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
