dexonsmith requested changes to this revision.
dexonsmith added a comment.
This revision now requires changes to proceed.
Herald added subscribers: kerbowa, steven_wu, atanasyan, zzheng, MaskRay,
fedor.sergeev, hiraditya, jvesely, sdardis, emaste.
Herald added a reviewer: espindola.
Herald added
mehdi_amini added a comment.
Thanks, I'll investigate function attributes then!
https://reviews.llvm.org/D24688
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
pcc added a comment.
In https://reviews.llvm.org/D24688#638836, @mehdi_amini wrote:
> In https://reviews.llvm.org/D24688#638835, @pcc wrote:
>
> > Didn't we figure out in the end that this could be a function attribute
> > instead?
>
>
> We did? You wrote in PR30403: "I had a brief look at what
hfinkel added a comment.
In https://reviews.llvm.org/D24688#639158, @jyknight wrote:
> Since this requires everyone generating llvm IR to add this module attribute
> for optimal codegen, it should be documented in release notes and the
> LangRef, I think.
And, for something like this, here:
mehdi_amini added a comment.
In https://reviews.llvm.org/D24688#639158, @jyknight wrote:
> Since this requires everyone generating llvm IR to add this module attribute
> for optimal codegen,
Not sure yet how it'll end up, I'll revamp this.
See also:
jyknight added a comment.
Since this requires everyone generating llvm IR to add this module attribute
for optimal codegen, it should be documented in release notes and the LangRef,
I think.
I mean: it's unfortunate that it's needed at all, but at the very least it
should be possible for
mehdi_amini added a comment.
In https://reviews.llvm.org/D24688#638835, @pcc wrote:
> Didn't we figure out in the end that this could be a function attribute
> instead?
We did? You wrote in PR30403: "I had a brief look at what it would take to have
a per-function TLI, and I'm not convinced
pcc added a comment.
Didn't we figure out in the end that this could be a function attribute instead?
https://reviews.llvm.org/D24688
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mehdi_amini updated this revision to Diff 83510.
mehdi_amini added a comment.
Rebase
https://reviews.llvm.org/D24688
Files:
clang/lib/CodeGen/BackendUtil.cpp
clang/lib/CodeGen/CodeGenModule.cpp
clang/test/CodeGen/nobuiltin.c
clang/test/CodeGenCUDA/flush-denormals.cu
pcc added a comment.
The word "hosted" takes its meaning from the C/C++ standards. It refers to a
build environment where the standard library is guaranteed to be available.
Note that we've been rethinking this approach on the mailing list, see
tstellarAMD added a comment.
What does "hosted mode" mean?
https://reviews.llvm.org/D24688
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
pcc created this revision.
pcc added reviewers: mehdi_amini, dexonsmith.
pcc added subscribers: llvm-commits, cfe-commits.
Herald added a reviewer: tstellarAMD.
Herald added subscribers: nhaehnle, nemanjai, mehdi_amini, jyknight, qcolombet.
This module flag is used to mark modules which were
12 matches
Mail list logo
