On Oct 24, 2012, at 9:25 PM, Chandler Carruth <[email protected]> wrote:
> On Wed, Oct 24, 2012 at 8:58 PM, Quentin Colombet <[email protected]> wrote: >> Hello, >> >> Following the adding of the new ForceSizeOpt attribute in LLVM (see commit >> r1664220), I am making a patch for clang to set this attribute on each >> function when the Oz optimization level is set. > > Can you fix the attribute based on the code review feedback first? I'd > rather not start generating the IR when it's about to change. Hi Chandler, I am not sure to know which review you are talking about. >From my understanding, according to Eli, it was ok, and according to Evan, it >was ok too but may change in the future. I was not making the future happening now, if it was that you meant :). Quentin > >> >> The idea is that optimizations will be able to adapt their behavior >> according to this attribute. In a second step, not given in this patch, it >> would be possible to make this attribute available for the end user, thus >> enabling code size optimization focus for specific functions. >> The patch also contains a test case that check that the attribute is set >> when compiled in Oz and not set otherwise. >> >> Quentin >> >> _______________________________________________ >> cfe-commits mailing list >> [email protected] >> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >> _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
