[PATCH] D20116: Add nosideeffects function attribute

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 16 14:19:39 PDT 2016

tstellarAMD added a comment.

In http://reviews.llvm.org/D20116#459637, @mehdi_amini wrote:

> In http://reviews.llvm.org/D20116#458920, @tstellarAMD wrote:
> > Ping.
> You're pinging but my impression is that it is waiting on you to address nhaehnle comment, did you miss it or did I miss an answer?

I didn't address it specifically, but I was planning to address it by waiting for http://reviews.llvm.org/D18714 to be committed first, since that patch has the correct test case.  I was actually hoping that pinging this patch would get people to look at http://reviews.llvm.org/D18714 too, since that's been outstanding for much longer.


More information about the llvm-commits mailing list