[cfe-commits] [PATCH] Oz optimization level sets ForceSizeOpt attribute for each function

Eric Christopher echristo at gmail.com
Mon Oct 29 13:21:39 PDT 2012

On Thu, Oct 25, 2012 at 8:46 AM, Quentin Colombet <qcolombet at apple.com> wrote:
> On Oct 24, 2012, at 9:25 PM, Chandler Carruth <chandlerc at google.com> wrote:
>> On Wed, Oct 24, 2012 at 8:58 PM, Quentin Colombet <qcolombet at apple.com> 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 :).

This is what he meant, what's the point of doing this as a short term
fix when the full fix is both a) not much more work and b) there are
objections to the existing patch?

You put a patch up for review and then didn't respond to the review
before you committed it.


More information about the cfe-commits mailing list