[PATCH] [PATCH][AArch64]Some ACLE pair builtin functions are not handled

Hao Liu Hao.Liu at arm.com
Mon Dec 9 18:01:51 PST 2013


  Hi David,

  You are right. I understand the meaning of code review from your comments.

  Actually, I believe this patch and patch in http://llvm-reviews.chandlerc.com/D2350 will fix the problems and won't introduce any new problems. Both cases are simple and easy to be fixed.

  But I originally thought it was a neccessary process to ask for a code review. And I thought I could commit the patches without approvement if the reviewers are too busy.

  Now I think I understand the meaning of code review. If the patch is very simple, maybe I can commit without code review. If I ask for code review, I should always wait the approvement.

  Thanks,
  -Hao

http://llvm-reviews.chandlerc.com/D2348



More information about the llvm-commits mailing list