lenary accepted this revision.
lenary added a comment.
This revision is now accepted and ready to land.

I'm happy with this patch, but you could update the description to be clearer 
that you're adding an option to allow PMU v3.4 to be enabled/disabled 
separately to the architecture - not adding the pmu extension itself.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116748

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to