anemet added a comment.
In http://reviews.llvm.org/D18489#384851, @davidxl wrote:
> What I meant is that putting the comments in InstrProfData.inc file, and
> update the one in CodeGenPGO.cpp to reference that.
Sounds good. You mean CGCall.cpp instead of CodeGenPGO.cpp though, correct?
So to
anemet accepted this revision.
anemet added a reviewer: anemet.
anemet added a comment.
This revision is now accepted and ready to land.
Was accepted by davidxl.
http://reviews.llvm.org/D18489
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
This revision was automatically updated to reflect the committed changes.
Closed by commit rL264678: [PGO] More comments how function pointers for
indirect calls are mapped (authored by anemet).
Changed prior to commit:
http://reviews.llvm.org/D18489?vs=51713&id=51845#toc
Repository:
rL LLVM
Author: anemet
Date: Mon Mar 28 17:18:53 2016
New Revision: 264681
URL: http://llvm.org/viewvc/llvm-project?rev=264681&view=rev
Log:
[PGO] More comments how function pointers for indirect calls are mapped
to function names
Summary:
Hopefully this will make it easier for the next person to figure
anemet added a comment.
The summary of commits:
llvm: http://reviews.llvm.org/rL264678
compiler-rt: http://reviews.llvm.org/rL264680
clang: http://reviews.llvm.org/rL264681
Repository:
rL LLVM
http://reviews.llvm.org/D18489
___
cfe-commits maili
> On Jan 18, 2016, at 5:28 PM, Hal Finkel wrote:
>
> - Original Message -
>> From: "Adam Nemet"
>> To: cfe-comm...@cs.uiuc.edu
>> Sent: Friday, January 16, 2015 12:51:50 PM
>> Subject: r226298 - [AVX512] Add intrinsics for masked aligned FP loads and
>> stores
>>
>> Author: anemet
>
Author: anemet
Date: Mon Jan 18 20:02:25 2016
New Revision: 258108
URL: http://llvm.org/viewvc/llvm-project?rev=258108&view=rev
Log:
[AVX512] Fix typo in r226298
Hal noticed that the double/float got mixed up on the parameters for
these.
Modified:
cfe/trunk/lib/Headers/avx512fintrin.h
Modif
> On Jan 18, 2016, at 6:00 PM, Adam Nemet wrote:
>
>
>> On Jan 18, 2016, at 5:28 PM, Hal Finkel wrote:
>>
>> - Original Message -
>>> From: "Adam Nemet"
>>> To: cfe-comm...@cs.uiuc.edu
>>> Sent: Friday, January 16, 2015 12:51:50 PM
>>> Subject: r226298 - [AVX512] Add intrinsics for m
@@ -659,13 +659,21 @@ llvm::ARM::FPUKind arm::getARMTargetFeatures(const Driver
&D,
CPUArgFPUKind != llvm::ARM::FK_INVALID ? CPUArgFPUKind :
ArchArgFPUKind;
(void)llvm::ARM::getFPUFeatures(FPUKind, Features);
} else {
+bool Generic = true;
if (!ForAS) {
anemet wrote:
We're also seeing LTO failures with this:
https://green.lab.llvm.org/job/llvm.org/job/clang-stage2-Rthinlto/900/
I think that this should be reverted. I also have a small reproducer but
unfortunately it requires the Apple linker.
https://github.com/llvm/llvm-project/pull/14110
anemet wrote:
> I have a fix for the crash that I'll send momentarily.
Do you have a link?
https://github.com/llvm/llvm-project/pull/141106
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com
101 - 111 of 111 matches
Mail list logo