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

I liked the idea of sorting these options at compile time.
I think the most dummy sort should work just fine inthis case, as the number of 
elements are not that large.
Do you think worth it?

Anyway , I think moving the runtime sort out of the ctor is a good 
middleground. Thank you.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137258

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

Reply via email to