Re: [PATCH] D42217: Set Module Metadata "AvoidPLT" when -fno-plt is used.

2018-02-23 Thread Sriraman Tallam via cfe-commits
On Fri, Feb 23, 2018 at 11:54 AM, Rafael Avila de Espindola wrote: > Sriraman Tallam via Phabricator writes: >> + if (CodeGenOpts.NoPLT) { >> +getModule().setAvoidPLT(); >> + } >> + > > You don't need the {}. > > LGTM with that.

Re: [PATCH] D42217: Set Module Metadata "AvoidPLT" when -fno-plt is used.

2018-02-23 Thread Rafael Avila de Espindola via cfe-commits
Sriraman Tallam via Phabricator writes: > + if (CodeGenOpts.NoPLT) { > +getModule().setAvoidPLT(); > + } > + You don't need the {}. LGTM with that. Cheers, Rafael ___ cfe-commits mailing list cfe-commits@lists.llvm.org