[PATCH] D96512: [PowerPC] Add option for ROP Protection

Amy Kwan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 16 13:38:25 PST 2021


amyk accepted this revision as: amyk.
amyk added a comment.
This revision is now accepted and ready to land.

I just have a small question about the patch but overall I think this LGTM.



================
Comment at: llvm/lib/Target/PowerPC/PPC.td:317
   // Power8
   list<SubtargetFeature> P8AdditionalFeatures =
     [DirectivePwr8,
----------------
Does `FeatureROPProtection` need to be here and in the P9/P10 features?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96512/new/

https://reviews.llvm.org/D96512



More information about the cfe-commits mailing list