<div dir="ltr">While doing the conversion of LLVM_OVERRIDE to 'override' last night, I noticed that the code base is rather inconsistent on whether the 'virtual' keyword is also used when 'override' is used.<div>
<br></div><div>Should we have a coding standard for this? What's the preferred direction here? Seems not having 'virtual' is less overall text, but not sure how others feel.</div><div><br></div><div>Related, should we require use of 'override' when methods override a base class method?</div>
<div><br></div><div>I have clang-tidy checks for both though haven't implemented fixits for them yet.<br clear="all"><div><br></div>Thanks,<br>~Craig
</div></div>